lkml.org 
[lkml]   [2015]   [Feb]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 3/3] tomoyo: robustify handling of mm->exe_file
From
Date
Thank you, but I think this patch is wrong and redundant.

Davidlohr Bueso wrote:
> On Wed, 2015-02-18 at 16:10 -0800, Davidlohr Bueso wrote:
> > +static const char *tomoyo_get_exe(struct mm_struct *mm)
> > +{
> > + struct file *exe_file;
> > + const char *cp = NULL;
> > +
> > + if (!mm)
> > + return NULL;
> > + exe_file = get_mm_exe_file(mm);
> > + if (!exe_file)
> > + return NULL;
> > +
> > + cp = tomoyo_realpath_from_path(&exe_file->f_path);
>
> tomoyo_realpath_from_path can return NULL here, thus we'd leak the
> f_path in the caller... I guess this should be:
>
> > + path_get(&exe_file->f_path);
>
> if (cp)
> path_get(&exe_file->f_path);
>
Why do we need to let the caller call path_put() ?
There is no need to do like proc_exe_link() does, for
tomoyo_get_exe() returns pathname as "char *".

> > + fput(exe_file);
> > + return cp;
> > +}


\
 
 \ /
  Last update: 2015-02-19 12:21    [W:0.174 / U:0.312 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site