On Thu, Aug 16, 2018 at 06:14:06PM +0800, Nan Xiao wrote:
> Hi tech@,
>
> The `free_out' variable seems redundant, so this patch removes it:
>
> Index: apps.c
> ===================================================================
> RCS file: /cvs/src/usr.bin/openssl/apps.c,v
> retrieving revision 1.47
> diff -u -p -r1.47 apps.c
> --- apps.c 7 Feb 2018 08:57:25 -0000 1.47
> +++ apps.c 16 Aug 2018 09:18:43 -0000
> @@ -2050,11 +2050,9 @@ policies_print(BIO *out, X509_STORE_CTX
> {
> X509_POLICY_TREE *tree;
> int explicit_policy;
> - int free_out = 0;
>
> if (out == NULL) {
> out = BIO_new_fp(stderr, BIO_NOCLOSE);
> - free_out = 1;
> }
> tree = X509_STORE_CTX_get0_policy_tree(ctx);
> explicit_policy = X509_STORE_CTX_get_explicit_policy(ctx);
>
> --
> Best Regards
> Nan Xiao
Committed. Thanks!
Rob
