[ovs-dev] [coverity 12/14] ovsdb: Don't check "date" before assignment in ovsdb_file_txn_from_json().

Ben Pfaff blp at nicira.com
Tue Feb 22 17:57:03 UTC 2011


On Mon, Feb 21, 2011 at 05:44:55PM -0800, Justin Pettit wrote:
> There's no indication that "date" is optional in the description of
> ovsdb_file_txn_from_json(), and the one caller always passes it in, so
> don't bother checking whether it exists.
> 
> Coverity #10732

Looks good, thanks.

Coverity probably noticed that we always did "*date = LLONG_MAX;" at the
top of this function, so indeed 'date' is not optional.




More information about the dev mailing list