Closed Bug 99419 Opened 23 years ago Closed 23 years ago

for loop executes only once -- CODE ERROR?

Categories

(NSS :: Libraries, defect, P1)

Product:
NSS
Component:
Libraries
Platform:
Sun
Solaris
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: timeless, Assigned: rrelyea)

References

(
URL
)

Details

from attachment.cgi 48274 to bug 98332,
-  for( i = 0; i < nSlots; i++ ) {
+  if ( nSlots > 0 ) {
+    i = 0;
     if( fwSlot == slots[i] ) {
       *pSlot = (CK_SLOT_ID)(CK_ULONG)(i+1);
     }
^^This change like all others merely removes warnings. But it's likely that a
for loop was chosen because the code was supposed to be executed more than once.
This is not the case -- as the warning indicates and as my equivalency rewrite
indicates.
Something is obviously wrong with that for loop.

Assigned the bug to Bob.
Assignee: wtc → relyea
Priority: -- → P1
Target Milestone: --- → 3.4
The return needed to happen within the 'if' statement. It should be fixed in
version 1.4 of wrap.c

bob
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.