[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
at-func2 (was: select, nanosleep, mbslen, at-func2)
From: |
Eric Blake |
Subject: |
at-func2 (was: select, nanosleep, mbslen, at-func2) |
Date: |
Wed, 06 Jan 2010 19:04:46 -0700 |
User-agent: |
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.23) Gecko/20090812 Thunderbird/2.0.0.23 Mnenhy/0.7.6.666 |
According to Ian Beckwith on 1/6/2010 6:05 PM:
>> during build of coreutils-8.1:
>>
>> at-func2.c: In function 'at_func2':
>> at-func2.c:113: warning: attempt to free a non-heap object 'proc_buf1'
>>
>> IMO, the right fix is
>>
>> --- lib/at-func2.c
>> +++ lib/at-func2.c
>> @@ -110,7 +110,7 @@
>> }
>> }
>> else if (proc_file1 != proc_buf1 && proc_file1 != file1)
>> - free (proc_buf1);
>> + free (proc_file1);
Ouch. Yes, that is the correct fix for my typo. Applied, in Petr's name.
> PS In general, would you prefer separate messages for each bug?
Yes, it is easier (at least for me) to do bookkeeping for outstanding bugs
if they are in separate threads. For example, I've just patched only 1 of
the three issues (across four interfaces) that you reported.
--
Don't work too hard, make some time for fun as well!
Eric Blake address@hidden
signature.asc
Description: OpenPGP digital signature
- select, nanosleep, mbslen, at-func2, Ian Beckwith, 2010/01/06
- at-func2 (was: select, nanosleep, mbslen, at-func2),
Eric Blake <=
- select (was: select, nanosleep, mbslen, at-func2), Eric Blake, 2010/01/06
- mbslen on MirBSD (was: select, nanosleep, mbslen, at-func2), Bruno Haible, 2010/01/07
- Prev by Date:
select, nanosleep, mbslen, at-func2
- Next by Date:
select (was: select, nanosleep, mbslen, at-func2)
- Previous by thread:
select, nanosleep, mbslen, at-func2
- Next by thread:
select (was: select, nanosleep, mbslen, at-func2)
- Index(es):