[ovs-dev] [PATCH] ovsdb: Catch negative timeout values

Ben Pfaff blp at nicira.com
Wed Aug 27 14:48:43 UTC 2014


On Wed, Aug 27, 2014 at 12:41:50AM +0200, Thomas Graf wrote:
> Although the check is present already, a missing !error branch
> in the next condition prevents the error from being reported back.
> 
> Signed-off-by: Thomas Graf <tgraf at noironetworks.com>

Are you sure?  I think that the only real bug here is that if the
timeout is negative *and* the "until" string is invalid, then there's a
memory leak of the error message for the negative timeout.  We should
still fix that, and the same code change does that, but I think that the
commit message is wrong.



More information about the dev mailing list