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

bugzilla at redhat.com bugzilla at redhat.com
Fri Apr 10 06:41:58 UTC 2015


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



--- Comment #589 from Anand Avati <aavati at redhat.com> ---
COMMIT: http://review.gluster.org/9582 committed in master by Vijay Bellur
(vbellur at redhat.com) 
------
commit ca9e4bf6bcc947dd7c179c93dae206c6cde4d3b5
Author: Nandaja Varma <nvarma at redhat.com>
Date:   Wed Feb 4 17:19:40 2015 +0530

    gluster-cli : Extra checks for word_count removed.

    Coverity IDs:
    1124453
    1124454

    In the first case, In the beginning of the function check is made
    to see of the word_count is less than 4. If yes it returns. So it
    wouldn't reach this part if that check takes the true path. So
    this code is logically dead.

    In second and third case, this is in false branch for word_cound <6.
    So word_count can't possibly be less than 5 at this point.

    Change-Id: Id5e1c81045ce7bc0ee6b7612ea54ef2b7f54b699
    BUG: 789278
    Signed-off-by: Nandaja Varma <nvarma at redhat.com>
    Reviewed-on: http://review.gluster.org/9582
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Humble Devassy Chirammal <humble.devassy at gmail.com>
    Reviewed-by: Vijay Bellur <vbellur at redhat.com>

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


More information about the Bugs mailing list