b0b627
# HG changeset patch
b0b627
# User Rob Lemley <rob@thunderbird.net>
b0b627
# Date 1663866557 14400
b0b627
#      Thu Sep 22 13:09:17 2022 -0400
b0b627
# Node ID 121afb4ed9b0e282cf6690736ffadf1498578434
b0b627
# Parent  0798506e89ab0ad98d5826effe2087c2e2560d0b
b0b627
Bug 1790116 - mozbuild changes for RNP v0.16.2. r=kaie
b0b627
hash_sha1cd.cpp moved up to its parent directory.
b0b627
b0b627
ENABLE_IDEA needs to be set to keep support enabled.
b0b627
https://github.com/rnpgp/rnp/commit/17972d0238919d4abf88b04debce95844be4716d
b0b627
b0b627
Update rnp_symbols.py to not include deprecated functions.
b0b627
Added new symbols to rnp.symbols for export.
b0b627
b0b627
Differential Revision: https://phabricator.services.mozilla.com/D157012
b0b627
b0b627
diff --git a/comm/python/thirdroc/thirdroc/rnp_symbols.py b/python/thirdroc/thirdroc/rnp_symb/commols.py
b0b627
--- a/comm/python/thirdroc/thirdroc/rnp_symbols.py
b0b627
+++ b/comm/python/thirdroc/thirdroc/rnp_symbols.py
b0b627
@@ -14,30 +14,75 @@ the third_party/rnp/include/rnp/rnp.h fo
b0b627
 Also note that APIs that are marked deprecated are not checked for.
b0b627
 
b0b627
 Dependencies: Only Python 3
b0b627
 
b0b627
 Running:
b0b627
-  python3 rnp_symbols.py
b0b627
+  python3 rnp_symbols.py [-h] [rnp.h path] [rnp.symbols path]
b0b627
 
b0b627
-Output will be on stdout, this is to give the developer the opportunity to compare the old and
b0b627
-new versions and check for accuracy.
b0b627
+Both file path arguments are optional. By default, the header file will be
b0b627
+read from "comm/third_party/rnp/include/rnp/rnp.h" and the symbols file will
b0b627
+be written to "comm/third_party/rnp/rnp.symbols".
b0b627
+
b0b627
+Path arguments are relative to the current working directory, the defaults
b0b627
+will be determined based on the location of this script.
b0b627
+
b0b627
+Either path argument can be '-' to use stdin or stdout respectively.
b0b627
 """
b0b627
 
b0b627
-from __future__ import absolute_import, print_function
b0b627
-
b0b627
+import argparse
b0b627
 import sys
b0b627
 import os
b0b627
 import re
b0b627
 
b0b627
 HERE = os.path.dirname(__file__)
b0b627
 TOPSRCDIR = os.path.abspath(os.path.join(HERE, "../../../../"))
b0b627
-RNPSRCDIR = os.path.join(TOPSRCDIR, "comm/third_party/rnp")
b0b627
+THIRD_SRCDIR = os.path.join(TOPSRCDIR, "comm/third_party")
b0b627
+HEADER_FILE_REL = "rnp/include/rnp/rnp.h"
b0b627
+HEADER_FILE = os.path.join(THIRD_SRCDIR, HEADER_FILE_REL)
b0b627
+SYMBOLS_FILE_REL = "rnp/rnp.symbols"
b0b627
+SYMBOLS_FILE = os.path.join(THIRD_SRCDIR, SYMBOLS_FILE_REL)
b0b627
 
b0b627
 
b0b627
 FUNC_DECL_RE = re.compile(r"^RNP_API\s+.*?([a-zA-Z0-9_]+)\(.*$")
b0b627
 
b0b627
 
b0b627
+class FileArg:
b0b627
+    """Based on argparse.FileType from the Python standard library.
b0b627
+    Modified to not open the filehandles until the open() method is
b0b627
+    called.
b0b627
+    """
b0b627
+
b0b627
+    def __init__(self, mode="r"):
b0b627
+        self._mode = mode
b0b627
+        self._fp = None
b0b627
+        self._file = None
b0b627
+
b0b627
+    def __call__(self, string):
b0b627
+        # the special argument "-" means sys.std{in,out}
b0b627
+        if string == "-":
b0b627
+            if "r" in self._mode:
b0b627
+                self._fp = sys.stdin.buffer if "b" in self._mode else sys.stdin
b0b627
+            elif "w" in self._mode:
b0b627
+                self._fp = sys.stdout.buffer if "b" in self._mode else sys.stdout
b0b627
+            else:
b0b627
+                raise ValueError(f"Invalid mode {self._mode} for stdin/stdout")
b0b627
+        else:
b0b627
+            if "r" in self._mode:
b0b627
+                if not os.path.isfile(string):
b0b627
+                    raise ValueError(f"Cannot read file {string}, does not exist.")
b0b627
+            elif "w" in self._mode:
b0b627
+                if not os.access(string, os.W_OK):
b0b627
+                    raise ValueError(f"Cannot write file {string}, permission denied.")
b0b627
+            self._file = string
b0b627
+        return self
b0b627
+
b0b627
+    def open(self):
b0b627
+        if self._fp:
b0b627
+            return self._fp
b0b627
+        return open(self._file, self._mode)
b0b627
+
b0b627
+
b0b627
 def get_func_name(line):
b0b627
     """
b0b627
     Extract the function name from a RNP_API function declaration.
b0b627
     Examples:
b0b627
     RNP_API rnp_result_t rnp_enable_debug(const char *file);
b0b627
@@ -46,24 +91,41 @@ def get_func_name(line):
b0b627
     """
b0b627
     m = FUNC_DECL_RE.match(line)
b0b627
     return m.group(1)
b0b627
 
b0b627
 
b0b627
-def extract_func_defs(filename):
b0b627
+def extract_func_defs(filearg):
b0b627
     """
b0b627
     Look for RNP_API in the header file to find the names of the symbols that should be exported
b0b627
     """
b0b627
-    with open(filename) as fp:
b0b627
+    with filearg.open() as fp:
b0b627
         for line in fp:
b0b627
-            if line.startswith("RNP_API"):
b0b627
+            if line.startswith("RNP_API") and "RNP_DEPRECATED" not in line:
b0b627
                 func_name = get_func_name(line)
b0b627
                 yield func_name
b0b627
 
b0b627
 
b0b627
 if __name__ == "__main__":
b0b627
-    if len(sys.argv) > 1:
b0b627
-        FILENAME = sys.argv[1]
b0b627
-    else:
b0b627
-        FILENAME = os.path.join(RNPSRCDIR, "include/rnp/rnp.h")
b0b627
+    parser = argparse.ArgumentParser(
b0b627
+        description="Update rnp.symbols file from rnp.h",
b0b627
+        epilog="To use stdin or stdout pass '-' for the argument.",
b0b627
+    )
b0b627
+    parser.add_argument(
b0b627
+        "header_file",
b0b627
+        default=HEADER_FILE,
b0b627
+        type=FileArg("r"),
b0b627
+        nargs="?",
b0b627
+        help=f"input path to rnp.h header file (default: {HEADER_FILE_REL})",
b0b627
+    )
b0b627
+    parser.add_argument(
b0b627
+        "symbols_file",
b0b627
+        default=SYMBOLS_FILE,
b0b627
+        type=FileArg("w"),
b0b627
+        nargs="?",
b0b627
+        help=f"output path to symbols file (default: {SYMBOLS_FILE_REL})",
b0b627
+    )
b0b627
 
b0b627
-    for f in sorted(list(extract_func_defs(FILENAME))):
b0b627
-        print(f)
b0b627
+    args = parser.parse_args()
b0b627
+
b0b627
+    with args.symbols_file.open() as out_fp:
b0b627
+        for symbol in sorted(list(extract_func_defs(args.header_file))):
b0b627
+            out_fp.write(f"{symbol}\n")
b0b627
diff --git a/comm/third_party/rnp/moz.build b/third_party/rnp/moz.b/commuild
b0b627
--- a/comm/third_party/rnp/moz.build
b0b627
+++ b/comm/third_party/rnp/moz.build
b0b627
@@ -41,10 +41,11 @@ rnp_defines = {
b0b627
     "HAVE_ZLIB_H": True,
b0b627
     "CRYPTO_BACKEND_BOTAN": True,
b0b627
     "ENABLE_AEAD": True,
b0b627
     "ENABLE_TWOFISH": True,
b0b627
     "ENABLE_BRAINPOOL": True,
b0b627
+    "ENABLE_IDEA": True,
b0b627
     "PACKAGE_BUGREPORT": '"https://bugzilla.mozilla.org/enter_bug.cgi?product=Thunderbird"',
b0b627
     "PACKAGE_STRING": '"rnp {}"'.format(CONFIG["MZLA_LIBRNP_FULL_VERSION"])
b0b627
 }
b0b627
 GeneratedFile(
b0b627
     "src/lib/config.h",
b0b627
@@ -119,16 +120,16 @@ SOURCES += [
b0b627
     "src/lib/crypto/ecdsa.cpp",
b0b627
     "src/lib/crypto/eddsa.cpp",
b0b627
     "src/lib/crypto/elgamal.cpp",
b0b627
     "src/lib/crypto/hash.cpp",
b0b627
     "src/lib/crypto/hash_common.cpp",
b0b627
+    "src/lib/crypto/hash_sha1cd.cpp",
b0b627
     "src/lib/crypto/mem.cpp",
b0b627
     "src/lib/crypto/mpi.cpp",
b0b627
     "src/lib/crypto/rng.cpp",
b0b627
     "src/lib/crypto/rsa.cpp",
b0b627
     "src/lib/crypto/s2k.cpp",
b0b627
-    "src/lib/crypto/sha1cd/hash_sha1cd.cpp",
b0b627
     "src/lib/crypto/sha1cd/sha1.c",
b0b627
     "src/lib/crypto/sha1cd/ubc_check.c",
b0b627
     "src/lib/crypto/signatures.cpp",
b0b627
     "src/lib/crypto/symmetric.cpp",
b0b627
     "src/lib/fingerprint.cpp",
b0b627
diff --git a/comm/third_party/rnp/rnp.symbols b/third_party/rnp/rnp.symb/commols
b0b627
--- a/comm/third_party/rnp/rnp.symbols
b0b627
+++ b/comm/third_party/rnp/rnp.symbols
b0b627
@@ -37,10 +37,11 @@ rnp_import_keys
b0b627
 rnp_import_signatures
b0b627
 rnp_input_destroy
b0b627
 rnp_input_from_callback
b0b627
 rnp_input_from_memory
b0b627
 rnp_input_from_path
b0b627
+rnp_input_from_stdin
b0b627
 rnp_key_25519_bits_tweak
b0b627
 rnp_key_25519_bits_tweaked
b0b627
 rnp_key_add_uid
b0b627
 rnp_key_allows_usage
b0b627
 rnp_key_export
b0b627
@@ -75,10 +76,11 @@ rnp_key_get_uid_count
b0b627
 rnp_key_get_uid_handle_at
b0b627
 rnp_key_handle_destroy
b0b627
 rnp_key_have_public
b0b627
 rnp_key_have_secret
b0b627
 rnp_key_is_compromised
b0b627
+rnp_key_is_expired
b0b627
 rnp_key_is_locked
b0b627
 rnp_key_is_primary
b0b627
 rnp_key_is_protected
b0b627
 rnp_key_is_retired
b0b627
 rnp_key_is_revoked
b0b627
@@ -112,10 +114,11 @@ rnp_op_encrypt_set_cipher
b0b627
 rnp_op_encrypt_set_compression
b0b627
 rnp_op_encrypt_set_creation_time
b0b627
 rnp_op_encrypt_set_expiration_time
b0b627
 rnp_op_encrypt_set_file_mtime
b0b627
 rnp_op_encrypt_set_file_name
b0b627
+rnp_op_encrypt_set_flags
b0b627
 rnp_op_encrypt_set_hash
b0b627
 rnp_op_generate_add_pref_cipher
b0b627
 rnp_op_generate_add_pref_compression
b0b627
 rnp_op_generate_add_pref_hash
b0b627
 rnp_op_generate_add_usage
b0b627
@@ -169,10 +172,11 @@ rnp_op_verify_get_signature_at
b0b627
 rnp_op_verify_get_signature_count
b0b627
 rnp_op_verify_get_symenc_at
b0b627
 rnp_op_verify_get_symenc_count
b0b627
 rnp_op_verify_get_used_recipient
b0b627
 rnp_op_verify_get_used_symenc
b0b627
+rnp_op_verify_set_flags
b0b627
 rnp_op_verify_signature_get_handle
b0b627
 rnp_op_verify_signature_get_hash
b0b627
 rnp_op_verify_signature_get_key
b0b627
 rnp_op_verify_signature_get_status
b0b627
 rnp_op_verify_signature_get_times
b0b627
@@ -185,21 +189,24 @@ rnp_output_to_armor
b0b627
 rnp_output_to_callback
b0b627
 rnp_output_to_file
b0b627
 rnp_output_to_memory
b0b627
 rnp_output_to_null
b0b627
 rnp_output_to_path
b0b627
+rnp_output_to_stdout
b0b627
 rnp_output_write
b0b627
 rnp_recipient_get_alg
b0b627
 rnp_recipient_get_keyid
b0b627
 rnp_remove_security_rule
b0b627
 rnp_request_password
b0b627
 rnp_result_to_string
b0b627
 rnp_save_keys
b0b627
+rnp_set_timestamp
b0b627
 rnp_signature_get_alg
b0b627
 rnp_signature_get_creation
b0b627
 rnp_signature_get_expiration
b0b627
 rnp_signature_get_hash_alg
b0b627
+rnp_signature_get_key_fprint
b0b627
 rnp_signature_get_keyid
b0b627
 rnp_signature_get_signer
b0b627
 rnp_signature_get_type
b0b627
 rnp_signature_handle_destroy
b0b627
 rnp_signature_is_valid