Remove unnecessary NUL terminators

All the string-copying functions called above do terminate the strings
they create with a NUL byte.  Writing it again at the end of the buffer
is unnecessary paranoid code.  Let's remove it.

Signed-off-by: Alejandro Colomar <alx@kernel.org>

diff --git a/src/groupmod.c b/src/groupmod.c
index 3799235cda632e36d84f8ba9ef10cc22c1604b27..ee6c3e6c1c242a3c405c9faa79558a92cab63c0b 100644 (file)
--- a/src/groupmod.c
+++ b/src/groupmod.c
@@ -627,11 +627,6 @@ static void prepare_failure_reports (void)
                stpeprintf(info_passwd.action+strlen (info_passwd.action),
                                 pw_end, "%ju", (uintmax_t) group_newid);
        }
-       info_group.audit_msg[511]   = '\0';
-#ifdef SHADOWGRP
-       info_gshadow.audit_msg[511] = '\0';
-#endif
-       info_passwd.audit_msg[511]  = '\0';
 
 // FIXME: add a system cleanup
        add_cleanup (cleanup_report_mod_group, &info_group);