LGTM with comment. https://codereview.appspot.com/262630043/diff/1/lib/pki/pkibase.c File lib/pki/pkibase.c (right): https://codereview.appspot.com/262630043/diff/1/lib/pki/pkibase.c#newcode1062 lib/pki/pkibase.c:1062: return (nssPKIObjectCollection *)NULL; You don't need to cast NULL here or below.