Closed Bug 454419 Opened 12 years ago Closed 12 years ago
prbool bugs in security/
The only thing I'm not sure about is the rv return fix in prbool func in nsPKCS12Blob.cpp as the original's logic is weird due to rv/prbool confusion.
(In reply to comment #0) > The only thing I'm not sure about is the rv return fix in prbool func in > nsPKCS12Blob.cpp as the original's logic is weird due to rv/prbool confusion. I agree. It seems all active code completely ignores the return value of function handleError(). Only some disabled code tried to look at it. I propose to change handleError() to have return type "void" and change the return statements to simply "return;"
That sounds good, could you do that change? Changing return statements is easy. I'm not familiar with the code an unsure of what the logic should for places that do check handleError's return value.
There are no places where the return value of handleError gets checked.
You are right, I was confusing it with some unrelated handleError
Kai, I'm waiting on your review so I can commit this.
Attachment #337973 - Flags: review?(kaie) → review+
Comment on attachment 337973 [details] [diff] [review] fixes r=kaie Please also delete the unnecessary declaration of variable keepGoing. Thanks a lot for the patch!
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.