Wireshark-commits: [Wireshark-commits] master 38d39f2: Have MemoryCheck() directly incorporate the
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 12 Jun 2016 19:18:57 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=38d39f292e01551fe30b57583bd79a0d06d59420
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

38d39f2 by Guy Harris (guy@xxxxxxxxxxxx):

    Have MemoryCheck() directly incorporate the error message and exit.
    
    Manually inline memory_error() in MemoryCheck(), so that static
    analyzers know that, if MemoryCheck() sees a null pointer, it exists,
    and the null pointer isn't subsequently used.
    
    Use MemoryCheck() instead of the one place where we manually checked for
    a null pointer and called memory_error().
    
    Change-Id: Id6b0328cfd17cb14ec9d1e461420896a31573c71
    Reviewed-on: https://code.wireshark.org/review/15851
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  7bbc60c   Add more memory checks, and use MemoryCheck() for some existing checks.
    adds  38d39f2   Have MemoryCheck() directly incorporate the error message and exit.


Summary of changes:
 tools/lemon/lemon.c |   16 +++-------------
 1 file changed, 3 insertions(+), 13 deletions(-)