diff options
author | Chong.Lu@windriver.com <Chong.Lu@windriver.com> | 2014-06-13 14:12:57 +0800 |
---|---|---|
committer | Andreas Oberritter <obi@opendreambox.org> | 2015-04-22 10:07:58 +0200 |
commit | f959e417e5014b313f2d927be61bb09204be43c3 (patch) | |
tree | acb1f14445f48a502016a8080dc4a092b96c7229 | |
parent | c5f159828a518f0b3482b687fe01d97d0f42b100 (diff) | |
download | meta-openembedded-contrib-f959e417e5014b313f2d927be61bb09204be43c3.tar.gz |
samba: Security Advisory - CVE-2013-4124
Integer overflow in the read_nttrans_ea_list function in nttrans.c in
smbd in Samba 3.x before 3.5.22, 3.6.x before 3.6.17, and 4.x before
4.0.8 allows remote attackers to cause a denial of service (memory
consumption) via a malformed packet.
http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2013-4124
Signed-off-by: Yue Tao <Yue.Tao@windriver.com>
Signed-off-by: Chong Lu <Chong.Lu@windriver.com>
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
(cherry picked from commit 69e765c4645f018506325312a42a53f984dbb6e8)
Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
-rw-r--r-- | meta-oe/recipes-connectivity/samba/samba/samba-3.6.16-CVE-2013-4124.patch | 43 | ||||
-rw-r--r-- | meta-oe/recipes-connectivity/samba/samba_3.6.8.bb | 1 |
2 files changed, 44 insertions, 0 deletions
diff --git a/meta-oe/recipes-connectivity/samba/samba/samba-3.6.16-CVE-2013-4124.patch b/meta-oe/recipes-connectivity/samba/samba/samba-3.6.16-CVE-2013-4124.patch new file mode 100644 index 0000000000..54b8edfbe6 --- /dev/null +++ b/meta-oe/recipes-connectivity/samba/samba/samba-3.6.16-CVE-2013-4124.patch @@ -0,0 +1,43 @@ +Upstream-Status: Backport + +From efdbcabbe97a594572d71d714d258a5854c5d8ce Mon Sep 17 00:00:00 2001 +From: Jeremy Allison <jra@samba.org> +Date: Wed, 10 Jul 2013 17:10:17 -0700 +Subject: [PATCH] Fix bug #10010 - Missing integer wrap protection in EA list + reading can cause server to loop with DOS. + +Ensure we never wrap whilst adding client provided input. +CVE-2013-4124 + +Signed-off-by: Jeremy Allison <jra@samba.org> +--- + source3/smbd/nttrans.c | 12 ++++++++++++ + 1 file changed, 12 insertions(+) + +diff --git a/source3/smbd/nttrans.c b/source3/smbd/nttrans.c +index ea9d417..5fc3a09 100644 +--- a/source3/smbd/nttrans.c ++++ b/source3/smbd/nttrans.c +@@ -989,7 +989,19 @@ struct ea_list *read_nttrans_ea_list(TALLOC_CTX *ctx, const char *pdata, size_t + if (next_offset == 0) { + break; + } ++ ++ /* Integer wrap protection for the increment. */ ++ if (offset + next_offset < offset) { ++ break; ++ } ++ + offset += next_offset; ++ ++ /* Integer wrap protection for while loop. */ ++ if (offset + 4 < offset) { ++ break; ++ } ++ + } + + return ea_list_head; +-- +1.7.10.4 + diff --git a/meta-oe/recipes-connectivity/samba/samba_3.6.8.bb b/meta-oe/recipes-connectivity/samba/samba_3.6.8.bb index a5e7dcd180..331796cb33 100644 --- a/meta-oe/recipes-connectivity/samba/samba_3.6.8.bb +++ b/meta-oe/recipes-connectivity/samba/samba_3.6.8.bb @@ -33,6 +33,7 @@ SRC_URI += "\ file://samba-3.6.22-CVE-2013-4496.patch;patchdir=.. \ file://0001-PIDL-fix-parsing-linemarkers-in-preprocessor-output.patch;patchdir=.. \ file://samba-3.6.11-CVE-2013-0213-CVE-2013-0214.patch;patchdir=.. \ + file://samba-3.6.16-CVE-2013-4124.patch;patchdir=.. \ " SRC_URI[md5sum] = "fbb245863eeef2fffe172df779a217be" SRC_URI[sha256sum] = "4f5a171a8d902c6b4f822ed875c51eb8339196d9ccf0ecd7f6521c966b3514de" |