[LTP] [PATCH 1/2] stack_clash: Guard functions used only in 64bit
    Petr Vorel 
    pvorel@suse.cz
       
    Thu Oct 26 01:14:48 CEST 2023
    
    
  
Hi Cyril,
> Hi!
> > To avoid "defined but not used" warnings when compiled on 32bit.
> > Signed-off-by: Petr Vorel <pvorel@suse.cz>
> > ---
> >  testcases/cve/stack_clash.c | 4 ++++
> >  1 file changed, 4 insertions(+)
> > diff --git a/testcases/cve/stack_clash.c b/testcases/cve/stack_clash.c
Hi Cyril,
> > +#ifdef __x86_64__
> >  static void force_bottom_up(void)
> >  {
> >  	FILE *fh;
> > @@ -134,6 +135,7 @@ static void force_bottom_up(void)
> >  out:
> >  	SAFE_FCLOSE(fh);
> >  }
> > +#endif
> Maybe I'm blind but this function does not seem to be called from inside
> of #ifdef __x86_64__.
force_bottom_up() is called only inside do_mmap_placement_test(), which is also
wrapped with #ifdef __x86_64_.
I did not notice that, compiler pointed it out :).
Can I merge with your RBT?
Kind regards,
Petr
    
    
More information about the ltp
mailing list