[Bugs] [Bug 789278] Issues reported by Coverity static analysis tool

bugzilla at redhat.com bugzilla at redhat.com
Mon Apr 10 15:19:28 UTC 2017


https://bugzilla.redhat.com/show_bug.cgi?id=789278



--- Comment #875 from Worker Ant <bugzilla-bot at gluster.org> ---
COMMIT: https://review.gluster.org/17026 committed in master by Shyamsundar
Ranganathan (srangana at redhat.com) 
------
commit 78c4e657dc2e122b00f99912819357c85e35d54d
Author: Niels de Vos <ndevos at redhat.com>
Date:   Mon Apr 10 11:02:41 2017 +0200

    gfapi: remove unneeded NULL-check in glfs_fsync_async_common()

    Coverity reported that "glfd" has a NULL-check at the end of the
    function, but is used without checking earlier. Because "glfd" is passed
    as parameter to the (internal) function, there is no need to have the
    additional NULL-check.

    Change-Id: I1390b2ef250fbf624f0bc2b0eda7b7588942aae9
    BUG: 789278
    Signed-off-by: Niels de Vos <ndevos at redhat.com>
    Reviewed-on: https://review.gluster.org/17026
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: Shyamsundar Ranganathan <srangana at redhat.com>

-- 
You are receiving this mail because:
You are the assignee for the bug.


More information about the Bugs mailing list