Revert "Revert "coredumpctl: in case of error free pattern after print""

In the case of the error set_consume will free the pointer 'pattern'
which is later used in log_error. Either we should stop priniting that
or use simple set_put and free pattern manually.

This reverts commit ece6b8fd5b.
This commit is contained in:
Lukas Nykryn 2014-01-15 12:46:44 +01:00
parent 3bedba4ae1
commit 1b2457e16d

View file

@ -126,10 +126,11 @@ static int add_match(Set *set, const char *match) {
goto fail;
log_debug("Adding pattern: %s", pattern);
r = set_consume(set, pattern);
r = set_put(set, pattern);
if (r < 0) {
log_error("Failed to add pattern '%s': %s",
pattern, strerror(-r));
free(pattern);
goto fail;
}