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

[bugzilla at redhat.com]

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



--- Comment #532 from Anand Avati <aavati at redhat.com> ---
COMMIT: http://review.gluster.org/10064 committed in master by Vijay Bellur
(vbellur at redhat.com) 
------
commit 5c1ce6d29ee52af9daa23c1c7a54f23ce29a687e
Author: Michael Adam <obnox at samba.org>
Date:   Tue Mar 31 10:30:22 2015 +0200

    features/trash: fix remove_trash_path() to do what it is intended.

    The intention is to set the the rem_path string pointer that
    is handed in from the outside. So the string has to be passed
    in by reference, or else the function just sets a pointer on
    the stack.

    Found by Coverity, CID 1288791

    Also fixes:
    CID 1288791
    CID 1288809
    CID 1288810
    CID 1288814
    CID 1288815
    CID 1288817
    CUD 1288818

    Change-Id: I6ef5eeb58b83a875cf90513291abe37dee3fc9fb
    BUG: 789278
    Signed-off-by: Michael Adam <obnox at samba.org>
    Reviewed-on: http://review.gluster.org/10064
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Anoop C S <achiraya at redhat.com>
    Reviewed-by: Vijay Bellur <vbellur at redhat.com>

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