Wireshark-commits: [Wireshark-commits] rev 18492: /trunk/wiretap/ /trunk/wiretap/: ber.c

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: sahlberg@xxxxxxxxxxxxx
Date: Sat, 17 Jun 2006 10:26:22 GMT
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=18492

User: sahlberg
Date: 2006/06/17 10:26 AM

Log:
 fix coverity "bug" 184   (false positive)
 
 
 the contract for wiretap ..._read() functions are that the *err pointer MUST always
 be provided as a valid pointer and not a NULL pointer.
 
 
 ber.c contained a spurios and redundant check for the pointer being non-NULL wich triggered coverity to detect this as a false positive.
 
 this redundant test if the pointer is NULL has been revoved.

Directory: /trunk/wiretap/
  Changes    Path          Action
  +1 -2      ber.c         Modified