Closed Bug 884178 Opened 12 years ago Closed 11 years ago

[patch] add PK11_CipherFinal macro

Categories

(NSS :: Libraries, enhancement, P2)

Product:
NSS
Component:
Libraries
Platform:
x86_64
Linux
Type:
enhancement

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
3.15.2

People

(Reporter: mbartos, Assigned: mbartos)

Details

Attachments

(1 file, 1 obsolete file)

add PK11_CipherFinal - #define
777 bytes, patch
rrelyea
: review+
Details | Diff | Splinter Review
Attached file add_PK11_CipherFinal.patch (obsolete) —
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:21.0) Gecko/20100101 Firefox/21.0 (Beta/Release) Build ID: 20130516103852 Steps to reproduce: This patch adds function wrapper PK11_CipherFinal for consistency when using PK11_CipherOp.
Attachment #763989 - Attachment is obsolete: true
Comment on attachment 765235 [details] [diff] [review] add PK11_CipherFinal - #define r+ for the change to pk11pub.h r- for the change to nss.def the #define means the application has PK11_CipherFinal in the code, but the linker will see PK11_DigestFinal.
Attachment #765235 - Flags: review+
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Keywords: checkin-needed
Assignee: nobody → nobody
Component: General → Libraries
Product: Core → NSS
Version: Trunk → trunk
Kai, can you land this please? :)
Flags: needinfo?(kaie)
> can you land this please? done https://hg.mozilla.org/projects/nss/rev/a47bc2881ba4
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Flags: needinfo?(kaie)
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 3.15.2
Patch failed to build on most platforms, only Linux builds worked. Backed out: https://hg.mozilla.org/projects/nss/rev/62facccdaaaf
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Please look at build logs available at https://nss-crypto.org:8011/waterfall
Kai, only the header file needs to be checked in. I r-'ed the nss.def change as not necessary (actually wrong).
Checked in header change. https://hg.mozilla.org/projects/nss/rev/804af8ec3e1f
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
It may be better to add PK11_CipherFinal as a real function, so that one can set a breakpoint for PK11_CipherFinal in a debugger without looking at the header file.
It may be better if there are some development guideline for this kind of stuff.
wtc, actually Milan proposed an actual function, I prefered the #define. it's far from the only #define to alias names in NSS... bob
Updated the Summary field to reflect what was actually added.
Assignee: nobody → mbartos
Severity: normal → enhancement
Priority: -- → P2
Summary: [patch] add PK11_CipherFinal function wrapper → [patch] add PK11_CipherFinal macro
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: