[lkml]   [2018]   [Mar]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: 98f929b1bd ("ipc/shm: Fix shmctl(..., IPC_STAT, ...) between .."): Oops: 0000 [#1]
Nagarathnam Muthusamy <> writes:

> Hi Eric,
>     From
> It looks like if the following condition in Line 616 succeeds
> error = PTR_ERR(file);
> if (IS_ERR(file))
>     goto no_file;
> we get to no_file with garbage value in shm_cprid. An attempt to
> put_pid on this garbage value might be causing panic.
> We could initialize shm_cprid to NULL as soon as it was created.

Yes. I misread the kvmalloc as a kvzalloc.
I am planning on placing the pid freeing under the no_id label
instead of the no_file label. Which should also avoid the issue.

It is a rare enough issue an incremental patch should be fine.


 \ /
  Last update: 2018-03-28 19:55    [W:2.281 / U:0.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site