397825 - libpkix: ifdef code that uses user object types

Status: RESOLVED FIXED

(NSS :: Libraries, enhancement, P2)

Description

[Alexei Volkov]

libpkix associates unique number(object type) with each structures defined by the library. There are two ways for creation of a new types: one - by adding new element into PKIX_TYPENUM enum(defined in pkixt.h); two - create new, so called, "user type" by calling PKIX_PL_Object_RegisterType.

Since current implementation does not use any "user types" functionality we should ifdef all code related to "user types".

Comment 1

[Alexei Volkov]

Attachment: Patch v1

This has been attached to the bug 391457 and reviewed by Nelson.

Comment 2

[Alexei Volkov]

Attachment: Patch v2

Comment 3

[Alexei Volkov]

Attachment: Patch v2

modified according to review comments.

Comment 5

[Nelson Bolyard (seldom reads bugmail)]

Comment on attachment 288763 [details] [diff] [review]
Patch v2

This patch creates many new places where we see code sequences like this:
 ...
 goto cleanup;
} else {
 ...
}
cleanup:

In those sequences, the goto cleanup is unnecessary.  But let's get this checked in and we can clean that up later.

While reviewing this patch, I noticed more structure declarations in header files, for structures that should be completely private, known only inside the related .c file.  We should fix all those to mitigate the temptation to access internals of other objects' private internals.  But we can do that in a later patch for another bug.

Comment 6

[Nelson Bolyard (seldom reads bugmail)]

Alexei, one more question:
Have you built this code (with the patch applied) both with and without 
the feature test macro PKIX_USER_OBJECT_TYPE defined?
Do you know for certain that this code still builds when that macro is 
defined?  
If it does not, then before we can close this bug we need to fix that.

Comment 7

[Alexei Volkov]

Tree builds with and without macro PKIX_USER_OBJECT_TYPE been defined.

Comment 8

[Alexei Volkov]

Patch v2 has been integrated.