diff options
author | Jo-Philipp Wich <jo@mein.io> | 2020-02-26 16:36:16 +0100 |
---|---|---|
committer | Jo-Philipp Wich <jo@mein.io> | 2020-02-26 16:37:27 +0100 |
commit | 817e77531942e6cc541e6cc92c6eb3655fe63b23 (patch) | |
tree | 6978543cc2419ec218ba39afee5e7f3362f37c02 /package/network/services/ppp/patches/700-radius-Prevent-buffer-overflow-in-rc_mksid.patch | |
parent | 8f93c05a591bd68e4d8eaa0a8468ce2263762004 (diff) | |
download | upstream-817e77531942e6cc541e6cc92c6eb3655fe63b23.tar.gz upstream-817e77531942e6cc541e6cc92c6eb3655fe63b23.tar.bz2 upstream-817e77531942e6cc541e6cc92c6eb3655fe63b23.zip |
Revert "ppp: backport security fixes"
This reverts commit 215598fd03899c19a9cd26266221269dd5ec8cee since it
didn't contain a reference to the CVE it addresses. The next commit
will re-add the commit including a CVE reference in its commit message.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
Diffstat (limited to 'package/network/services/ppp/patches/700-radius-Prevent-buffer-overflow-in-rc_mksid.patch')
-rw-r--r-- | package/network/services/ppp/patches/700-radius-Prevent-buffer-overflow-in-rc_mksid.patch | 30 |
1 files changed, 0 insertions, 30 deletions
diff --git a/package/network/services/ppp/patches/700-radius-Prevent-buffer-overflow-in-rc_mksid.patch b/package/network/services/ppp/patches/700-radius-Prevent-buffer-overflow-in-rc_mksid.patch deleted file mode 100644 index 1c5722e6dd..0000000000 --- a/package/network/services/ppp/patches/700-radius-Prevent-buffer-overflow-in-rc_mksid.patch +++ /dev/null @@ -1,30 +0,0 @@ -From 858976b1fc3107f1261aae337831959b511b83c2 Mon Sep 17 00:00:00 2001 -From: Paul Mackerras <paulus@ozlabs.org> -Date: Sat, 4 Jan 2020 12:01:32 +1100 -Subject: [PATCH] radius: Prevent buffer overflow in rc_mksid() - -On some systems getpid() can return a value greater than 65535. -Increase the size of buf[] to allow for this, and use slprintf() -to make sure we never overflow it. - -Signed-off-by: Paul Mackerras <paulus@ozlabs.org> ---- - pppd/plugins/radius/util.c | 4 ++-- - 1 file changed, 2 insertions(+), 2 deletions(-) - -diff --git a/pppd/plugins/radius/util.c b/pppd/plugins/radius/util.c -index 6f976a712951..740131e8377c 100644 ---- a/pppd/plugins/radius/util.c -+++ b/pppd/plugins/radius/util.c -@@ -73,9 +73,9 @@ void rc_mdelay(int msecs) - char * - rc_mksid (void) - { -- static char buf[15]; -+ static char buf[32]; - static unsigned short int cnt = 0; -- sprintf (buf, "%08lX%04X%02hX", -+ slprintf(buf, sizeof(buf), "%08lX%04X%02hX", - (unsigned long int) time (NULL), - (unsigned int) getpid (), - cnt & 0xFF); |