Le 11/03/2015 09:29, Martin Pitt a écrit :
Hello all,
Didier Roche [2015-03-10 17:56 +0100]:
--- a/src/fsckd/fsckd.c
+++ b/src/fsckd/fsckd.c
@@ -272,7 +272,7 @@ static int plymouth_send_message(int plymouth_fd, const
char *message, bool upda
}
static int manager_send_plymouth_message(Manager *m, const char *message) {
- const char *plymouth_cancel_message = NULL;
+ _cleanup_free_ const char *plymouth_cancel_message = NULL,
*l10n_cancel_message = NULL;
int r;
As far as I can see, this would free(l10n_cancel_message) on exit, but
you must never free the result of gettext(). So split this into
_cleanup_free_ const char *plymouth_cancel_message = NULL;
const char *l10n_cancel_message;
Indeed (weird I didn't get a double free crash), but the man page says
it's statically allocated. Will fix it and report while bringing up the
other patch with the architecture modification.
Thanks!
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel
