Split secp256k1_ecmult_gen_context_teardown() out of secp256k1_ecmult_gen_context_clear

The 'teardown' naming is to complement the 'build' function name.  To 'build'
allocates and sets up the memory for ctx->prec. To 'teardown' frees the memory.

Author: isle2983

src/ecmult_gen.h

@@ -32,6 +32,9 @@ static void secp256k1_ecmult_gen_context_init(secp256k1_ecmult_gen_context* ctx)
 static void secp256k1_ecmult_gen_context_build(secp256k1_ecmult_gen_context* ctx, const secp256k1_callback* cb);
 static void secp256k1_ecmult_gen_context_clone(secp256k1_ecmult_gen_context *dst,
                                                const secp256k1_ecmult_gen_context* src, const secp256k1_callback* cb);
+#ifndef USE_ECMULT_STATIC_PRECOMPUTATION
+static void secp256k1_ecmult_gen_context_teardown(secp256k1_ecmult_gen_context* ctx);
+#endif
 static void secp256k1_ecmult_gen_context_clear(secp256k1_ecmult_gen_context* ctx);
 static int secp256k1_ecmult_gen_context_is_built(const secp256k1_ecmult_gen_context* ctx);
 

src/ecmult_gen_impl.h

@@ -112,10 +112,13 @@ static void secp256k1_ecmult_gen_context_clone(secp256k1_ecmult_gen_context *dst
     }
 }
 
-static void secp256k1_ecmult_gen_context_clear(secp256k1_ecmult_gen_context *ctx) {
 #ifndef USE_ECMULT_STATIC_PRECOMPUTATION
+static void secp256k1_ecmult_gen_context_teardown(secp256k1_ecmult_gen_context *ctx) {
     free(ctx->prec);
+}
 #endif
+
+static void secp256k1_ecmult_gen_context_clear(secp256k1_ecmult_gen_context *ctx) {
     SECP256K1_CLEANSE(ctx->blind);
     SECP256K1_CLEANSE(ctx->initial);
     ctx->prec = NULL;

src/gen_context.c

@@ -64,6 +64,9 @@ int main(int argc, char **argv) {
         }
     }
     fprintf(fp,"};\n");
+#ifndef USE_ECMULT_STATIC_PRECOMPUTATION
+    secp256k1_ecmult_gen_context_teardown(&ctx);
+#endif
     secp256k1_ecmult_gen_context_clear(&ctx);
 
     fprintf(fp, "#undef SC\n");

src/secp256k1.c

@@ -93,6 +93,9 @@ void secp256k1_context_destroy(secp256k1_context* ctx) {
     if (ctx != NULL) {
         secp256k1_ecmult_context_teardown(&ctx->ecmult_ctx);
         SECP256K1_CLEANSE(ctx->ecmult_ctx);
+#ifndef USE_ECMULT_STATIC_PRECOMPUTATION
+        secp256k1_ecmult_gen_context_teardown(&ctx->ecmult_gen_ctx);
+#endif
         secp256k1_ecmult_gen_context_clear(&ctx->ecmult_gen_ctx);
 
         free(ctx);