Wireshark-commits: [Wireshark-commits] master a76c6cf: lemon: Fix Dereference of null pointer found
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 30 Dec 2016 00:56:42 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a76c6cfa429b4dd204b14999d27187faa61d0541
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

a76c6cf by Alexis La Goutte (alexis.lagoutte@xxxxxxxxx):

    lemon: Fix Dereference of null pointer found by Clang analyzer
    
    lemon.c:877:10: warning: Access to field 'lhs' results in a dereference of a null pointer (loaded from field 'startRule')
    lemon.c:1141:22: warning: Access to field 'lhs' results in a dereference of a null pointer (loaded from field 'startRule')
    
    it is false positve lem.nrule==0 implies lem.rule==NULL
    
    Change-Id: Id63086990762fbf5195bce34a28f25aeb7a246f7
    Reviewed-on: https://code.wireshark.org/review/19445
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  255722c   Fix Interface Selection List
    adds  a76c6cf   lemon: Fix Dereference of null pointer found by Clang analyzer


Summary of changes:
 tools/lemon/lemon.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)