Closed
Bug 1561587
Opened 5 years ago
Closed 5 years ago
Remove -Wmaybe-uninitialized warning in lgattr.c
Categories
(NSS :: Build, defect, P2)
NSS
Build
Tracking
(Not tracked)
RESOLVED
FIXED
3.46
People
(Reporter: giulio.benetti, Assigned: giulio.benetti)
Details
Attachments
(1 file)
883 bytes,
patch
|
kjacobs
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/74.0.3729.169 Safari/537.36
Steps to reproduce:
make nss_build_all BUILD_OPT=1 ALLOW_OPT_CODE_SIZE=1 OPT_CODE_SIZE=1
but using -Og flag instead of -Os.
Actual results:
lgattr.c: In function ‘lg_FindTrustAttribute’:
lgattr.c:1072:11: error: ‘crv’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
CK_RV crv;
^~~
Expected results:
No warning.
Assignee | ||
Comment 1•5 years ago
|
||
This patch fixes warning.
Updated•5 years ago
|
Assignee: nobody → giulio.benetti
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Priority: -- → P2
Updated•5 years ago
|
Attachment #9074162 -
Flags: review?(kjacobs.bugzilla)
Comment 2•5 years ago
|
||
Comment on attachment 9074162 [details] [diff] [review]
0001-Bug-1561587-Remove-Wmaybe-uninitialized-warning-in-l.patch
Review of attachment 9074162 [details] [diff] [review]:
-----------------------------------------------------------------
Thank you for the patch!
Attachment #9074162 -
Flags: review?(kjacobs.bugzilla) → review+
Updated•5 years ago
|
Keywords: checkin-needed
Comment 3•5 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 3.46
You need to log in
before you can comment on or make changes to this bug.
Description
•