lib/freebl/prng_fips1861.c: unused variable `A'

RESOLVED FIXED in 3.9.1

Status

NSS
Libraries
P4
normal
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: Wan-Teh Chang, Assigned: Nelson Bolyard (seldom reads bugmail))

Tracking

3.9.1
PowerPC
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

753 bytes, patch
Nelson Bolyard (seldom reads bugmail)
: review+
Details | Diff | Splinter Review
(Reporter)

Description

14 years ago
When compiling lib/freebl/prng_fips1861.c on Mac OS X,
gcc issues the following warning:

cc -o Darwin6.8_DBG.OBJ/prng_fips1861.o -c -g  -Dppc -Wmost -fpascal-strings
-no-cpp-precomp -fno-common -pipe -DDARWIN -DHAVE_STRERROR -DHAVE_BSD_FLOCK
-DXP_UNIX -DSHLIB_SUFFIX=\"dylib\" -DSHLIB_PREFIX=\"lib\"
-DRIJNDAEL_INCLUDE_TABLES -DDEBUG -UNDEBUG -DDEBUG_wtchang -DMP_API_COMPATIBLE
-I../../../../dist/Darwin6.8_DBG.OBJ/include  -I../../../../dist/public/nss
-I../../../../dist/private/nss -Impi -Iecl  prng_fips1861.c
prng_fips1861.c: In function `RNG_UpdateAndEnd_FIPS186_1':
prng_fips1861.c:441: warning: unused variable `A'
(Reporter)

Comment 1

14 years ago
Created attachment 137901 [details] [diff] [review]
Proposed patch

I believe this is the exact condition under which
the local variable 'A' is used.  Please verify.
(Assignee)

Updated

14 years ago
Priority: -- → P4
(Assignee)

Comment 2

14 years ago
Will fix for 3.9.1
Status: NEW → ASSIGNED
Target Milestone: --- → 3.9.1
(Assignee)

Comment 3

14 years ago
Comment on attachment 137901 [details] [diff] [review]
Proposed patch

r=MisterSSL.  I confirm.
Wan-Teh, do you want this bug reasigned to you to check it in??
Attachment #137901 - Flags: review+
(Reporter)

Comment 4

14 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.