On Wed, Mar 27, 2013 at 03:16:35PM +0100, Vaclav Pavlin wrote: > From: Václav Pavlín <[email protected]> > > --- > src/core/execute.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/src/core/execute.c b/src/core/execute.c > index bc876a3..936e7d8 100644 > --- a/src/core/execute.c > +++ b/src/core/execute.c > @@ -1559,10 +1559,16 @@ void exec_context_tmp_dirs_done(ExecContext *c) { > > for(dirp = dirs; *dirp; dirp++) { > char *dir; > - rm_rf_dangerous(*dirp, false, true, false); > + int r=0; > > + r = rm_rf_dangerous(*dirp, false, true, false); > dir = dirname(*dirp); > - rmdir(dir); > + if (r<0) > + log_warning("Failed to remove content of tmp_dir > %s.", dir); > + else { > + dir = dirname(*dirp); > + rmdir(dir); > + } > > free(*dirp); > } Applied this one with a tweak to check rmdir return value too.
Zbyszek _______________________________________________ systemd-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/systemd-devel
