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

bugzilla at redhat.com bugzilla at redhat.com
Sun Feb 26 16:34:40 UTC 2017


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



--- Comment #845 from Worker Ant <bugzilla-bot at gluster.org> ---
COMMIT: https://review.gluster.org/16758 committed in master by Shyamsundar
Ranganathan (srangana at redhat.com) 
------
commit 7fcc85748c54687a1a29f25db7155beccba9d186
Author: Michael Scherer <misc at redhat.com>
Date:   Fri Feb 24 21:35:38 2017 +0100

    mgmt: Reset conf_fd to default value to avoid double close

    Coverity warn of a path where we use sys_close 2 times on the
    same file descriptor, which is likely harmless but could
    cause various hard to debug problems if threads are used
    (since the file descriptor table is shared among all
    threads, we could close a newly opened fd by another thread).

    Change-Id: I0524b31dccc0da94c7b87583e2a88ef06e003518
    BUG: 789278
    Signed-off-by: Michael Scherer <misc at redhat.com>
    Reviewed-on: https://review.gluster.org/16758
    Tested-by: Michael Scherer <misc at fedoraproject.org>
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    CentOS-regression: 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