New subject: [Bug 517722] tog-pegasus-2.9.0-1.7: possible off by one error in for loop

30 Jun 2009


      http://bugzilla.novell.com/show_bug.cgi?id=517722
Summary: tog-pegasus-2.9.0-1.7: possible off by one error in
                    for loop
    Classification: openSUSE
           Product: openSUSE 11.2
           Version: Factory
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Normal
          Priority: P5 - None
         Component: Other
        AssignedTo: bnc-team-screening@forge.provo.novell.com
        ReportedBy: dcb314@hotmail.com
         QAContact: qa@suse.de
          Found By: ---
I just had a look at factory package tog-pegasus-2.9.0-1.7
For source code file pegasus/src/Pegasus/Common/Logger.cpp,
around line 258 is the source code
for (unsigned int i=0; i<=timeStamp.size(); i++)
            {
                if(timeStamp[i] == '/' || timeStamp[i] == ':')
                {
Clearly broken code. Suggest new code
for (unsigned int i = 0; i < timeStamp.size(); i++)
-- 
Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Bug 517722] New: tog-pegasus-2.9.0-1.7: possible off by one error in for loop