lkml.org 
[lkml]   [2009]   [Jan]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] ftrace: unsigned idx cannot be less than 0

[ added Liming to CC ]

On Fri, 2 Jan 2009, Frederic Weisbecker wrote:

> On Fri, Jan 02, 2009 at 03:49:43PM +0100, Roel Kluin wrote:
> > // vi kernel/trace/ftrace.c +787
> > struct ftrace_iterator {
> > ...
> > unsigned idx;
> > ...
> > };
> >
> > idx is unsigned and cannot be less than 0.
> >
> > Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
> > ---
> > diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c
> > index 2f32969..a344add 100644
> > --- a/kernel/trace/ftrace.c
> > +++ b/kernel/trace/ftrace.c
> > @@ -842,7 +842,7 @@ static void *t_start(struct seq_file *m, loff_t *pos)
> > void *p = NULL;
> >
> > if (*pos > 0) {
> > - if (iter->idx < 0)
> > + if (iter->idx == 0)
> > return p;
> > (*pos)--;
> > iter->idx--;
>
>
> Hi Roel,
>
> I'm not sure this is the right fix.
> If you look at t_next, if there is no more page to look at,
> iter_idx takes -1.
>
> A 0 value would mean: we are in the first index on the page, which means
> there is something to read and we don't want to return NULL.
>
> I guess that would be better to turn idx into a signed int.

Correct. This bug was added by:

50cdaf08a8ec1d7f43987705da7aff7cf949708f
ftrace: improve seq_operation of ftrace

So the correct fix is to turn it into a signed int.

Thanks,

-- Steve


\
 
 \ /
  Last update: 2009-01-06 16:53    [W:2.175 / U:0.220 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site