[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Suspicious conditional

From: Dmitry <wipedout_at_yandex.ru>
Date: Wed, 29 Jun 2011 11:19:01 +0400

Hey.

Currently CPicture::SetActiveFrame() reads:

UINT prevframe = frame;
prevframe--;
if (prevframe < 0)
     prevframe = 0;

here prefframe is unsigned, so it can't be < 0. This defect was detected using PVS-Studio static analysis tool.

Best wishes.
Dmitry.

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2780438

To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-06-29 09:19:07 CEST

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.