Source-Changes-D archive

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

Re: CVS commit: src/sys/dev/bluetooth



What about the M_PREPEND()?

Le 29/04/2015 23:07, Iain Hibbert a écrit :
On Wed, 29 Apr 2015, Christos Zoulas wrote:

On Apr 29,  4:27pm, max%M00nBSD.net@localhost (Maxime Villard) wrote:
-- Subject: Re: CVS commit: src/sys/dev/bluetooth

| You didn't test this change, nor did you test the five other fixes
| you committed.

I checked the code to the best of my ability; perhaps I missed something.

by the way, the Brainy code scanner did not pick them up (perhaps it stops
at the first one?) but when I looked briefly at this code, there seem
other calls to bcsp_tx_unreliable_pkt() which also leak the mbuf on
failure, since if it fails it does not free it?

iain



Home | Main Index | Thread Index | Old Index