1224240 - fix memory leak in Bluetooth*Manager::CompareHeaderTarget

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Bluetooth, defect)

Description

[Nathan Froyd [:froydnj]]

ObexHeaderSet::GetTarget copies the data associated with the
ObexHeaderId::Target id into a newly-allocated buffer.  All callers of
this function, however, fail to free the buffer once they are done with it.

Instead of simply freeing the buffer in the caller, however, let's add
an API to ObexHeaderSet that gives direct access to the desired header.
Doing this means that we have direct access to the data--no copying
necessary--and we also make the caller slightly faster, since it no
longer has to verify that the appropriate header is there, followed by
re-searching for the header it already knows is there.

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: fix memory leak in Bluetooth*Manager::CompareHeaderTarget

Comment 2

[Ben Tian (inactive)]

Comment on attachment 8686644 [details] [diff] [review]
fix memory leak in Bluetooth*Manager::CompareHeaderTarget

Review of attachment 8686644 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with comment addressed.

::: dom/bluetooth/common/ObexBase.h
@@ +298,5 @@
>      // The specified parameter don't exist in 'Application Parameters' header.
>      return false;
>    }
>  
> +  const ObexHeader* HeaderOrNull(ObexHeaderId aId) const

Rename to |GetHeader|.

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/94dc62e0323e

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/94dc62e0323e