diff options
| author | Samanta Navarro <ferivoz@riseup.net> | 2024-01-23 12:04:05 +0000 |
|---|---|---|
| committer | Serge Hallyn <serge@hallyn.com> | 2024-03-14 17:20:30 -0500 |
| commit | 2b67dc77651874999258c9ca4ebd57ed03cabe37 (patch) | |
| tree | 052559a9d20b88af0b7e1e68db4bf5c9588acda1 | |
| parent | fce1d884795f311c1024915fc4a9161e141fcebd (diff) | |
lib/pam_pass_non_interactive.c: use strzero/free
The combination of bzero and free could be optimized away.
Reviewed-by: Alejandro Colomar <alx@kernel.org>
Signed-off-by: Samanta Navarro <ferivoz@riseup.net>
| -rw-r--r-- | lib/pam_pass_non_interactive.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/pam_pass_non_interactive.c b/lib/pam_pass_non_interactive.c index 12dad4dd..3d10908b 100644 --- a/lib/pam_pass_non_interactive.c +++ b/lib/pam_pass_non_interactive.c @@ -19,6 +19,7 @@ #include "alloc.h" #include "attr.h" +#include "memzero.h" #include "prototypes.h" #include "shadowlog.h" @@ -97,9 +98,8 @@ static int ni_conv (int num_msg, failed_conversation: for (count=0; count < num_msg; count++) { if (NULL != responses[count].resp) { - bzero(responses[count].resp, - strlen(responses[count].resp)); - free (responses[count].resp); + strzero(responses[count].resp); + free(responses[count].resp); responses[count].resp = NULL; } } |