Quoting Nathan Lynch ([email protected]):
> "Serge E. Hallyn" <[email protected]> writes:
> > Signed-off-by: Serge Hallyn <[email protected]>
> > ---
> >  arch/powerpc/kernel/vdso.c |    4 +++-
> >  1 files changed, 3 insertions(+), 1 deletions(-)
> >
> > diff --git a/arch/powerpc/kernel/vdso.c b/arch/powerpc/kernel/vdso.c
> > index 48beff6..679b394 100644
> > --- a/arch/powerpc/kernel/vdso.c
> > +++ b/arch/powerpc/kernel/vdso.c
> > @@ -240,8 +240,10 @@ int arch_setup_additional_pages(struct linux_binprm 
> > *bprm,
> >     }
> >  
> >     /* for restart(2), double check that we got we asked for */
> > -   if (start && vdso_base != start)
> > +   if (start && vdso_base != start) {
> > +           rc = -EBUSY;
> >             goto fail_mmapsem;
> > +   }
> >  
> >     /*
> >      * our vma flags don't have VM_WRITE so by default, the process isn't
> 
> Is this why you sent this patch?
> 
> cc1: warnings being treated as errors
> arch/powerpc/kernel/vdso.c: In function arch_setup_additional_pages:
> arch/powerpc/kernel/vdso.c:194: error: rc may be used uninitialized in this 
> function

Yup.

> If so, looks fine to me.

-serge
_______________________________________________
Containers mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/containers

_______________________________________________
Devel mailing list
[email protected]
https://openvz.org/mailman/listinfo/devel

Reply via email to