Closed Bug 530907 Opened 15 years ago Closed 15 years ago

The peerID argument to SSL_SetSockPeerID should be declared const

Categories

(NSS :: Libraries, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
3.12.6

People

(Reporter: wtc, Assigned: wtc)

Details

Attachments

(1 file)

Attached patch Proposed patchSplinter Review
The prototype of the SSL_SetSockPeerID function is: SECStatus SSL_SetSockPeerID(PRFileDesc *fd, char *peerID) The peerID argument should be declared const. The attached patch also removes an unnecessary PR_CALLBACK qualifier.
Attachment #414368 - Flags: review?(alexei.volkov.bugs)
Attachment #414368 - Flags: review?(alexei.volkov.bugs) → review+
Thank you, Nelson. I checked in the patch on the NSS trunk (NSS 3.12.6). Checking in ssl.h; /cvsroot/mozilla/security/nss/lib/ssl/ssl.h,v <-- ssl.h new revision: 1.31; previous revision: 1.30 done Checking in sslsock.c; /cvsroot/mozilla/security/nss/lib/ssl/sslsock.c,v <-- sslsock.c new revision: 1.60; previous revision: 1.59 done
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: