[SCM] Samba Shared Repository - branch master updated
Andrew Bartlett
abartlet at samba.org
Thu Sep 11 00:51:05 MDT 2014
The branch, master has been updated
via 01d4c0e torture: Provide enough space for test EA name in raw.eas test
from 1c00533 ctdb-build: Remove unnecessary lib/util include path from tests
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master
- Log -----------------------------------------------------------------
commit 01d4c0e804e0c10fe6acfbdbe6747dcc5df6abc9
Author: Andrew Bartlett <abartlet at samba.org>
Date: Tue Sep 2 14:38:11 2014 +1200
torture: Provide enough space for test EA name in raw.eas test
The issue is that previously bad_ea_name[5] was the last element on
the array, and so when we later did a strlen() on it, we read past the
end of the stack array. We need bad_ea_name[5] to be the second-last
element, followed by the \0 placed there by the strlcpy().
Found by AddressSanitizer
Change-Id: I871c08200aa2591c612dfa44da92b83132f83d88
Signed-off-by: Andrew Bartlett <abartlet at samba.org>
Reviewed-by: Kamen Mazdrashki <kamenim at samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet at samba.org>
Autobuild-Date(master): Thu Sep 11 08:50:16 CEST 2014 on sn-devel-104
-----------------------------------------------------------------------
Summary of changes:
source4/torture/raw/eas.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
Changeset truncated at 500 lines:
diff --git a/source4/torture/raw/eas.c b/source4/torture/raw/eas.c
index 95a55d1..15bfb2f 100644
--- a/source4/torture/raw/eas.c
+++ b/source4/torture/raw/eas.c
@@ -51,7 +51,7 @@ static bool test_eas(struct smbcli_state *cli, struct torture_context *tctx)
union smb_open io;
const char *fname = BASEDIR "\\ea.txt";
bool ret = true;
- char bad_ea_name[6];
+ char bad_ea_name[7];
int i;
int fnum = -1;
--
Samba Shared Repository
More information about the samba-cvs
mailing list