Wireshark-commits: [Wireshark-commits] master 483daa4: checkAPIs.pl: check value_string termination
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: Wed, 29 Mar 2017 20:11:37 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=483daa4d276968e152ac991323c9274c4cbf40ac
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

483daa4 by Peter Wu (peter@xxxxxxxxxxxxx):

    checkAPIs.pl: check value_string termination
    
    Check for the expected "{0, NULL}" termination, disallowing things like
    "{9, NULL}". Also add val64_string and bytes_string while at it. Remove
    a stale comment since we allow some c99 syntax now.
    
    Change-Id: I1685255bbb11d84eba46b7cd00934d2f74187e03
    Reviewed-on: https://code.wireshark.org/review/20792
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  4d1060e   mrcpv2: fix value_string termination
    adds  483daa4   checkAPIs.pl: check value_string termination


Summary of changes:
 tools/checkAPIs.pl |   22 ++++++++++------------
 1 file changed, 10 insertions(+), 12 deletions(-)