Message ID | 1401283300-14176-7-git-send-email-arei.gonglei@huawei.com |
---|---|
State | New |
Headers | show |
On Wed, May 28, 2014 at 3:21 PM, <arei.gonglei@huawei.com> wrote: > From: Gonglei <arei.gonglei@huawei.com> > > Spotted by Coverity: > > (8) Event freed_arg: "fclose(FILE *)" frees "wav->f". > (9) Event cond_true: Condition "fclose(wav->f)", taking true branch > Also see events: [pass_freed_arg] > > 212 if (fclose (wav->f)) { > (10) Event pass_freed_arg: Passing freed pointer "wav->f" as an argument > to function "AUD_log(char const *, char const *, ...)". > Also see events: [freed_arg] > > 213 dolog ("wav_fini_out: fclose %p failed\nReason: %s\n", > 214 wav->f, strerror (errno)); > > Removed wav->f's pointer in error log, actually it's uselessly. > > Signed-off-by: Gonglei <arei.gonglei@huawei.com> > Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> > --- > audio/wavaudio.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/audio/wavaudio.c b/audio/wavaudio.c > index 6846a1a..9bbe8e9 100644 > --- a/audio/wavaudio.c > +++ b/audio/wavaudio.c > @@ -210,8 +210,8 @@ static void wav_fini_out (HWVoiceOut *hw) > > doclose: > if (fclose (wav->f)) { > - dolog ("wav_fini_out: fclose %p failed\nReason: %s\n", > - wav->f, strerror (errno)); > + dolog ("wav_fini_out: fclose 'wav->f' failed\nReason: %s\n", > + strerror (errno)); This patch mainly just silences Coverity. Printing the old pointer value is a bit useless but doesn't pose a problem. Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
diff --git a/audio/wavaudio.c b/audio/wavaudio.c index 6846a1a..9bbe8e9 100644 --- a/audio/wavaudio.c +++ b/audio/wavaudio.c @@ -210,8 +210,8 @@ static void wav_fini_out (HWVoiceOut *hw) doclose: if (fclose (wav->f)) { - dolog ("wav_fini_out: fclose %p failed\nReason: %s\n", - wav->f, strerror (errno)); + dolog ("wav_fini_out: fclose 'wav->f' failed\nReason: %s\n", + strerror (errno)); } wav->f = NULL;