On Wed, 2018-11-28 at 23:06 +0300, Vitaly Chikunov wrote: > @@ -1773,6 +1776,7 @@ static char *get_password(void) > int main(int argc, char *argv[]) > { > int err = 0, c, lind; > + ENGINE *eng = NULL; > > g_argv = argv; > g_argc = argc; > @@ -1883,6 +1887,18 @@ int main(int argc, char *argv[]) > case 138: > measurement_list = 1; > break; > + case 139: /* --engine e */ > + eng = ENGINE_by_id(optarg); The usage is only adding "--engine e" support. Either change the usage or add a test to verify the argument. > + if (!eng) { > + log_err("engine %s isn't available\n", optarg); > + ERR_print_errors_fp(stderr); > + } else if (!ENGINE_init(eng)) { > + log_err("engine %s init failed\n", optarg); > + ERR_print_errors_fp(stderr); > + ENGINE_free(eng); > + eng = NULL; > + } > + break; > case 140: /* --xattr-user */ > xattr_ima = "user.ima"; > xattr_evm = "user.evm";