Comment 18 for bug 1429427

Revision history for this message
Eric Desrochers (slashd) wrote :

[ STS SPONSOR REVIEW ]

@Seyeong,

According to your dediff you are only cherry-picking "7a2e247 bugfix: plug a memleak in imuxsock".

This commit has been introduced upstream starting with "v7.6.3-158-g7a2e247", thus it seems like this bug is only affecting Trusty as you previously mentioned.

According to package version Xenial/Zesty and Artful already have the fix as they are > than "v7.6.3".

# Ubuntu
 rsyslog | 7.4.4-1ubuntu2.6 | trusty-updates | source, amd64, arm64, armhf, i386, powerpc, ppc64el
 rsyslog | 7.4.4-1ubuntu14 | vivid | source, amd64, arm64, armhf, i386, powerpc, ppc64el
 rsyslog | 8.16.0-1ubuntu3 | xenial | source, amd64, arm64, armhf, i386, powerpc, ppc64el, s390x
 rsyslog | 8.16.0-1ubuntu5 | yakkety | source, amd64, arm64, armhf, i386, powerpc, ppc64el, s390x
 rsyslog | 8.16.0-1ubuntu5 | zesty | source, amd64, arm64, armhf, i386, ppc64el, s390x
 rsyslog | 8.16.0-1ubuntu5 | artful | source, amd64, arm64, armhf, i386, ppc64el, s390x

and also aleady in Debian :

# Debian
 rsyslog | 8.24.0-1 | stable | source, amd64, arm64, armel, armhf, i386, mips, mips64el, mipsel, ppc64el, s390x
 rsyslog | 8.28.0-1 | testing | source, amd64, arm64, armel, armhf, i386, mips, mips64el, mipsel, ppc64el, s390x
 rsyslog | 8.28.0-1 | unstable | source, amd64, arm64, armel, armhf, i386, mips, mips64el, mipsel, powerpc, ppc64el, s390x
 rsyslog | 8.28.0-1 | unstable-debug | source

Looking at the upstream project I see 3 commits as follow for the same subject : "bugfix: plug a memleak in imuxsock"

$ git log --oneline --grep="plug a memleak in imuxsock"
be45099 bugfix: plug a memleak in imuxsock

#git describe be45099
v7.6.3-159-gbe45099

7a2e247 bugfix: plug a memleak in imuxsock

# git describe 7a2e247
v7.6.3-158-g7a2e247

5a11a1d bugfix: plug a memleak in imuxsock

# git describe 5a11a1d
v7.6.3-119-g5a11a1d

Does the other commits are needed to have a sane rsyslog package ?

I just want to double-check with you that we are not missing anything before uploading this package in the upload queue.

- Eric