21 Jan
2010
21 Jan
'10
10:51 p.m.
This patch makes it compile, thanks. :)
Acked-by: Andrew Morgan ziltro@ziltro.com
--
Andrew.
Michael Karcher wrote:
> recent gcc/glibc combinations warn about ignoring the fgets() result.
> The problem exists on Ubuntu 9.10 with current updates. This "fix" of
> the non-problem (as I check ferror() afterwards) should even be a
> (negligible) performance optimization.
>
> Signed-off-by: Michael Karcher flashrom@mkarcher.dialup.fu-berlin.de
> ---
> dmi.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/dmi.c b/dmi.c
> index ca75461..d165e9f 100644
> --- a/dmi.c
> +++ b/dmi.c
> @@ -79,8 +79,8 @@ void dmi_init(void)
> printf_debug("DMI pipe open error\n");
> goto out_free;
> }
> - fgets(answerbuf, DMI_MAX_ANSWER_LEN, dmidecode_pipe);
> - if (ferror(dmidecode_pipe))
> + if (!fgets(answerbuf, DMI_MAX_ANSWER_LEN, dmidecode_pipe) &&
> + ferror(dmidecode_pipe))
> {
> printf_debug("DMI pipe read error\n");
> pclose(dmidecode_pipe);
>