[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 #874 from Worker Ant <bugzilla-bot at gluster.org> ---
COMMIT: https://review.gluster.org/17023 committed in master by Jeff Darcy
(jeff at pl.atyp.us) 
------
commit 88551d8567503b3eba17e54fbe7dd9ee6266c1ab
Author: Niels de Vos <ndevos at redhat.com>
Date:   Mon Apr 10 10:10:30 2017 +0200

    upcall: correct the cleanup path in init() when GF_CALLOC() fails

    Coverity found that "priv" can be NULL when the 1st GF_CALLOC() fails,
    and still is being used with "priv->xattrs" before the functon returns.

    While cleaning this up, also removing the log message for out of memory,
    as this is already logged through GF_CALLOC().

    BUG: 789278
    Change-Id: I887eaa9136dc25a39c107cd2152b1ba11f9fa925
    Signed-off-by: Niels de Vos <ndevos at redhat.com>
    Reviewed-on: https://review.gluster.org/17023
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    NetBSD-regression: NetBSD Build System <jenkins at build.gluster.org>
    Reviewed-by: soumya k <skoduri at redhat.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: Amar Tumballi <amarts at redhat.com>

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


More information about the Bugs mailing list