| commit | 01d4c0e804e0c10fe6acfbdbe6747dcc5df6abc9 | |
| author | Andrew Bartlett <abartlet@samba.org> | |
| Tue, 2 Sep 2014 02:38:11 +0000 (2 14:38 +1200) | ||
| committer | Andrew Bartlett <abartlet@samba.org> | |
| Thu, 11 Sep 2014 06:50:16 +0000 (11 08:50 +0200) | ||
| tree | 59dfb32be13a3d9f5eef867a56ab50a55d1156c7 | treesnapshot (tar.gz zip) |
| parent | 1c00533bc0695117cc234d272ec3e0c8c19d3932 | commitdiff |
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@samba.org>
Reviewed-by: Kamen Mazdrashki <kamenim@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu Sep 11 08:50:16 CEST 2014 on sn-devel-104
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@samba.org>
Reviewed-by: Kamen Mazdrashki <kamenim@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu Sep 11 08:50:16 CEST 2014 on sn-devel-104
| source4/torture/raw/eas.c | diffblobblamehistory |