RE: [PATCH] staging: android: ion: Fixed uninitialized heap name access

2018-10-23 Thread Skidanov, Alexey
> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Tuesday, October 23, 2018 08:33 > To: Skidanov, Alexey > Cc: Laura Abbott ; gre...@linuxfoundation.org; > de...@driverdev.osuosl.org > Subject: Re: [PATCH] staging: android: ion

Re: [PATCH] staging: android: ion: Fixed uninitialized heap name access

2018-10-22 Thread Dan Carpenter
On Mon, Oct 22, 2018 at 05:47:08PM +0300, Alexey Skidanov wrote: > > > On 10/22/18 17:32, Laura Abbott wrote: > > On 10/22/2018 07:02 AM, Alexey Skidanov wrote: > >> The heap name might be uninitialized and access might crash the > >> kernel. > >> > > > > The heap name should never be null so

Re: [PATCH] staging: android: ion: Fixed uninitialized heap name access

2018-10-22 Thread Alexey Skidanov
On 10/22/18 17:32, Laura Abbott wrote: > On 10/22/2018 07:02 AM, Alexey Skidanov wrote: >> The heap name might be uninitialized and access might crash the >> kernel. >> > > The heap name should never be null so this seems like this is being > fixed in the wrong place. Can you explain more how

Re: [PATCH] staging: android: ion: Fixed uninitialized heap name access

2018-10-22 Thread Laura Abbott
On 10/22/2018 07:02 AM, Alexey Skidanov wrote: The heap name might be uninitialized and access might crash the kernel. The heap name should never be null so this seems like this is being fixed in the wrong place. Can you explain more how you are hitting this issue? Thanks, Laura

[PATCH] staging: android: ion: Fixed uninitialized heap name access

2018-10-22 Thread Alexey Skidanov
The heap name might be uninitialized and access might crash the kernel. Signed-off-by: Alexey Skidanov --- drivers/staging/android/ion/ion.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c index