söndag 30 december 2018 kl. 09:38:57 CET skrev Salvatore Bonaccorso: > There is an alternative approach wich was raised by Magnus in the > respective bug: https://bugs.debian.org/914632#12 (and see followup > from Moritz).
So, is it OK to upload this (assuming there's no code out there that actually uses SET_RSHPATH or SET_SSHPATH)? (By no longer defining RSHPATH, rshpath doesn't get set by the following code and tcp_aopen() will return NIL without doing anything. #ifdef RSHPATH /* rsh path defined yet? */ if (!rshpath) rshpath = cpystr (RSHPATH); #endif ) -- Magnus Holmgren holmg...@debian.org Debian Developer
commit e644cfbd0aac25ef320912e58e0d37cb715b2b88 Author: Magnus Holmgren <mag...@kibibyte.se> Date: Sun Jan 13 21:07:41 2019 +0100 [CVE-2018-19518] 2013_disable_rsh.patch (new): Disable access to IMAP mailboxes through running imapd over rsh (Closes: #914632). diff --git a/debian/changelog b/debian/changelog index b6960d1..cc7b885 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,12 @@ +uw-imap (8:2007f~dfsg-5+deb9u1) stretch-security; urgency=high + + * [CVE-2018-19518] 2013_disable_rsh.patch (new): Disable access to IMAP + mailboxes through running imapd over rsh, and therefore ssh (Closes: + #914632). Code using the library can enable it with tcp_parameters() + after making sure that the IMAP server name is sanitized. + + -- Magnus Holmgren <holmg...@debian.org> Sun, 24 Feb 2019 19:45:23 +0100 + uw-imap (8:2007f~dfsg-5) unstable; urgency=low * 1006_openssl1.1_autoverify.patch (new): Use new features for diff --git a/debian/patches/2013_disable_rsh.patch b/debian/patches/2013_disable_rsh.patch new file mode 100644 index 0000000..7a68644 --- /dev/null +++ b/debian/patches/2013_disable_rsh.patch @@ -0,0 +1,11 @@ +--- a/src/osdep/unix/Makefile ++++ b/src/osdep/unix/Makefile +@@ -985,7 +985,7 @@ onceenv: + -DMD5ENABLE=\"$(MD5PWD)\" -DMAILSPOOL=\"$(MAILSPOOL)\" \ + -DANONYMOUSHOME=\"$(MAILSPOOL)/anonymous\" \ + -DACTIVEFILE=\"$(ACTIVEFILE)\" -DNEWSSPOOL=\"$(NEWSSPOOL)\" \ +- -DRSHPATH=\"$(RSHPATH)\" -DLOCKPGM=\"$(LOCKPGM)\" \ ++ -DLOCKPGM=\"$(LOCKPGM)\" \ + -DLOCKPGM1=\"$(LOCKPGM1)\" -DLOCKPGM2=\"$(LOCKPGM2)\" \ + -DLOCKPGM3=\"$(LOCKPGM3)\" > OSCFLAGS + echo $(BASELDFLAGS) $(EXTRALDFLAGS) > LDFLAGS diff --git a/debian/patches/series b/debian/patches/series index ced3fcc..80d69e8 100644 --- a/debian/patches/series +++ b/debian/patches/series @@ -9,3 +9,4 @@ 2012_krb5_multidev.patch 1005_poll.patch 1006_openssl1.1_autoverify.patch +2013_disable_rsh.patch
signature.asc
Description: This is a digitally signed message part.