Re: [PATCH 1/1 v2] binder: free memory on error

2017-08-21 Thread Dan Carpenter
On Mon, Aug 21, 2017 at 04:13:28PM +0200, Christian Brauner wrote: > On binder_init() the devices string is duplicated and smashed into individual > device names which are passed along. However, the original duplicated string > wasn't freed in case binder_init() failed. Let's free it on error. >

Re: [PATCH 1/1 v2] binder: free memory on error

2017-08-21 Thread Dan Carpenter
On Mon, Aug 21, 2017 at 04:13:28PM +0200, Christian Brauner wrote: > On binder_init() the devices string is duplicated and smashed into individual > device names which are passed along. However, the original duplicated string > wasn't freed in case binder_init() failed. Let's free it on error. >

[PATCH 1/1 v2] binder: free memory on error

2017-08-21 Thread Christian Brauner
On binder_init() the devices string is duplicated and smashed into individual device names which are passed along. However, the original duplicated string wasn't freed in case binder_init() failed. Let's free it on error. Signed-off-by: Christian Brauner ---

[PATCH 1/1 v2] binder: free memory on error

2017-08-21 Thread Christian Brauner
On binder_init() the devices string is duplicated and smashed into individual device names which are passed along. However, the original duplicated string wasn't freed in case binder_init() failed. Let's free it on error. Signed-off-by: Christian Brauner --- Changelog v2 2017-08-21: - use more