Wireshark-commits: [Wireshark-commits] master 630b5a8: Check for localtime() failing.

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 29 Mar 2017 01:42:42 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=630b5a8165aab25d99989916f36412a0414068be
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

630b5a8 by Guy Harris (guy@xxxxxxxxxxxx):

    Check for localtime() failing.
    
    It "shouldn't happen", but at least this squelches a Coverity complaint,
    CID 1394503.
    
    Change-Id: I40af10d47c1d1b026f6b40ef68b139e6bf246109
    Reviewed-on: https://code.wireshark.org/review/20774
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  1bea950   Add bounds checks for references to outhdr_values.
    adds  630b5a8   Check for localtime() failing.


Summary of changes:
 text2pcap.c |   14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)