[infinite loop] bugs - nsPop3Protocol.cpp

VERIFIED FIXED in M10

Status

MailNews Core
Networking
P3
normal
VERIFIED FIXED
19 years ago
11 months ago

People

(Reporter: jay, Assigned: Scott MacGregor)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
Analysis of the code has determined possible infinite loop
or problem in the code below.  See bug
http://bugzilla.mozilla.org/show_bug.cgi?id=11588
for more information

  nsPop3Protocol.cpp:530: warning: unsigned value < 0 is always 0
  nsPop3Protocol.cpp:561: warning: unsigned value < 0 is always 0
  nsPop3Protocol.cpp:665: warning: unsigned value < 0 is always 0
  nsPop3Protocol.cpp:1027: warning: unsigned value < 0 is always 0
  nsPop3Protocol.cpp:1133: warning: unsigned value < 0 is always 0
  nsPop3Protocol.cpp:1301: warning: unsigned value < 0 is always 0
  nsPop3Protocol.cpp:1412: warning: unsigned value < 0 is always 0
(Reporter)

Updated

19 years ago
Blocks: 11588
Summary: [infinite loop] bugs - nsPop3Protocol.cpp → [infinite loop] bugs - nsPop3Protocol.cpp
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M10

Updated

19 years ago
Component: Browser-General → Networking-Mail
Product: Browser → MailNews
QA Contact: leger → lchiang

Comment 1

19 years ago
correcting component to mailnews.
(Assignee)

Comment 2

19 years ago
The line numbers are now out of date due to code modifications in the file. How
did we get these lines numbers in the first place? I couldn't find them in
slamm's warning page nor by looking at a linux build log. I ask, so I can try to
trigger them again to figure out the new line numbers I should look at. Thanks.
(Assignee)

Comment 3

19 years ago
Jay/Chris, how did you guys generate these warnings? This file has changed a lot
since the warnings were generated so the line numbers are all different. I
wanted to generate the warnings again so I could match it up against the current
contents of the file.

Comment 4

19 years ago
check bug 11588 for info on how to get a linux warning with the current line no.
use -W
(Assignee)

Comment 5

19 years ago
I've fixed these problems in nsPop3Protocol.cpp. I'll be checking it in today
when the tree opens so no need to triage this bug further.
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

19 years ago
I checked in the fix for this today.

Comment 7

19 years ago
Par, can you verify at your convenience?
QA Contact: lchiang → ppandit

Comment 8

18 years ago
VERIFIED based on code inspection of nsPop3Protocol.cpp.

Par
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.