[OE-core] [for-dora] systemd: journald fix ignored disk space restrictions
Diego Sueiro
diego.sueiro at gmail.com
Thu Feb 6 10:34:17 UTC 2014
Folks,
Any chance to get this up-streamed?
Regards,
--
*dS
Diego Sueiro
Administrador do Embarcados
www.embarcados.com.br
/*long live rock 'n roll*/
On Sat, Feb 1, 2014 at 7:07 AM, Diego Sueiro <diego.sueiro at gmail.com> wrote:
> The upstream bug report can be seen at:
> [Systemd #68161] -- https://bugs.freedesktop.org/show_bug.cgi?id=68161
>
> This backports patches come from 207 and need to address this in the 206
> version for dora branch.
>
> Signed-off-by: Diego Sueiro <diego.sueiro at gmail.com>
> ---
> ...ournald-fix-fd-leak-in-journal_file_empty.patch | 27 ++++++++++++++
> ...urnald-fix-vacuuming-of-archived-journals.patch | 37
> ++++++++++++++++++++
> meta/recipes-core/systemd/systemd_206.bb | 2 ++
> 3 files changed, 66 insertions(+)
> create mode 100644
> meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> create mode 100644
> meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
>
> diff --git
> a/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> new file mode 100644
> index 0000000..a83c858
> --- /dev/null
> +++
> b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> @@ -0,0 +1,27 @@
> +journald: fix fd leak in journal_file_empty
> +Before my previous patch, journal_file_empty wasn't be called with the
> +correct filename. Now that it's being called with the correct filename
> +it leaks file descriptors. This patch closes the file descriptors before
> +returning.
> +
> +Signed-off-by: George McCollister <george.mccollister at gmail.com>
> +
> +
> +Index: systemd-206/src/journal/journal-vacuum.c
> +===================================================================
> +--- systemd-206.orig/src/journal/journal-vacuum.c 2013-07-21
> 19:43:28.146183128 -0300
> ++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29
> 20:43:27.492343295 -0200
> +@@ -129,9 +129,9 @@
> + }
> +
> + static int journal_file_empty(int dir_fd, const char *name) {
> +- int fd, r;
> ++ int r;
> + le64_t n_entries;
> +-
> ++ _cleanup_close_ int fd;
> + fd = openat(dir_fd, name,
> O_RDONLY|O_CLOEXEC|O_NOFOLLOW|O_NONBLOCK);
> + if (fd < 0)
> + return -errno;
> +
> +Upstream-Status: Backport [207]
> diff --git
> a/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
> b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
> new file mode 100644
> index 0000000..7c59200
> --- /dev/null
> +++
> b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
> @@ -0,0 +1,37 @@
> +journald: fix vacuuming of archived journals
> +d_name is modified on line 227 so if the entire journal name is needed
> +again p must be used. Before this change when journal_file_empty was
> called
> +on archived journals it would always return with -2.
> +
> +Signed-off-by: George McCollister <george.mccollister at gmail.com>
> +
> +Index: systemd-206/src/journal/journal-vacuum.c
> +===================================================================
> +--- systemd-206.orig/src/journal/journal-vacuum.c 2014-01-29
> 20:43:27.492343295 -0200
> ++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29
> 20:48:55.076356876 -0200
> +@@ -265,18 +265,18 @@
> + /* We do not vacuum active files or unknown
> files! */
> + continue;
> +
> +- if (journal_file_empty(dirfd(d), de->d_name)) {
> ++ if (journal_file_empty(dirfd(d), p)) {
> +
> + /* Always vacuum empty non-online files. */
> +
> +- if (unlinkat(dirfd(d), de->d_name, 0) >= 0)
> +- log_debug("Deleted empty journal
> %s/%s.", directory, de->d_name);
> ++ if (unlinkat(dirfd(d), p, 0) >= 0)
> ++ log_debug("Deleted empty journal
> %s/%s.", directory, p);
> + else if (errno != ENOENT)
> +- log_warning("Failed to delete %s/%s:
> %m", directory, de->d_name);
> ++ log_warning("Failed to delete %s/%s:
> %m", directory, p);
> + continue;
> + }
> +
> +- patch_realtime(directory, de->d_name, &st, &realtime);
> ++ patch_realtime(directory, p, &st, &realtime);
> +
> + GREEDY_REALLOC(list, n_allocated, n_list + 1);
> +
> +
> +Upstream-Status: Backport [207]
> diff --git a/meta/recipes-core/systemd/systemd_206.bbb/meta/recipes-core/systemd/
> systemd_206.bb
> index a1a8bd7..e7d0f1f 100644
> --- a/meta/recipes-core/systemd/systemd_206.bb
> +++ b/meta/recipes-core/systemd/systemd_206.bb
> @@ -21,6 +21,8 @@ SRC_URI = "
> http://www.freedesktop.org/software/systemd/systemd-${PV}.tar.xz \
> file://0001-use-CAP_MKNOD-ConditionCapability.patch \
> file://0001-Use-bin-mkdir-instead-of-host-mkdir-path.patch \
> file://binfmt-install.patch \
> + file://journald-fix-fd-leak-in-journal_file_empty.patch \
> + file://journald-fix-vacuuming-of-archived-journals.patch \
> file://touchscreen.rules \
> ${UCLIBCPATCHES} \
> file://00-create-volatile.conf \
> --
> 1.7.10.4
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20140206/dcef90d0/attachment-0002.html>
More information about the Openembedded-core
mailing list