[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v2] symbols.c: Avoid warn_unused_result build failure on fgets().
On 11/09/15 11:06, Riku Voipio wrote: > In commit: > > d37d63d symbols: prefix static symbols with their source file names > > An unchecked fgets was added. This causes a compile error at least > on ubuntu utopic: > > symbols.c: In function 'read_symbol': > symbols.c:181:3: error: ignoring return value of 'fgets', declared with > attribute warn_unused_result [-Werror=unused-result] > fgets(str, 500, in); /* discard rest of line */ > ^ > > Paper over the warning by checking the return value in the if statement. > > Cc: Jan Beulich <jbeulich@xxxxxxxx> > Signed-off-by: Riku Voipio <riku.voipio@xxxxxxxxxx> Tested on Ubuntu 14.04.3 with GCC 4.8.4 and no 'warn_unused_result' warnings any more. Tested-by: Haozhong Zhang <haozhong.zhang@xxxxxxxxx> > --- > xen/tools/symbols.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/xen/tools/symbols.c b/xen/tools/symbols.c > index dbf6a1a..3b8fa39 100644 > --- a/xen/tools/symbols.c > +++ b/xen/tools/symbols.c > @@ -177,8 +177,8 @@ static int read_symbol(FILE *in, struct sym_entry *s) > rc = 0; > > skip_tail: > - if (input_format == fmt_sysv) > - fgets(str, 500, in); /* discard rest of line */ > + if ((input_format == fmt_sysv) && fgets(str, 500, in) == NULL) > + return rc; /* discard rest of line and ignore errors */ > > return rc; > } > -- > 2.6.2 > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@xxxxxxxxxxxxx > http://lists.xen.org/xen-devel _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |