Re: [PATCH BlueZ] Fix allocation of attribute values

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Hendrik,

On Fri, Sep 16, 2011 at 7:05 AM, Anderson Lizardo
<anderson.lizardo@xxxxxxxxxxxxx> wrote:
>>>                memcpy(a->data, value, vlen);
>>
>> And here you do...hmm...hard to tell since the pointer now shows to...where?
>> Even if that pointer is valid and is assigned the right size memory, you
>> just leaked memory.

After some thought (and thanks to Johan over IRC), I now see which
issue you are referring to. I'm reviewing all other places where this
bug might happen as well. Unfortunately, looks like it slipped through
manual review and valgrind altogether :(

Thanks,
-- 
Anderson Lizardo
Instituto Nokia de Tecnologia - INdT
Manaus - Brazil
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux