sin/samba-mirror
1
0
Fork 0
mirror of https://github.com/samba-team/samba.git synced 2025-01-08 21:18:16 +03:00
Code

lib/fuzzing: fuzz_sddl_parse: allow non-round-trip with long strings

Browse Source 
There is a borderline case where a conditional ACE unicode string
becomes longer than the SDDL parser wants to handle when control
characters are given canonical escaping. This can make the round trip
fail, but it isn't really a problem.

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
This commit is contained in:
Douglas Bagnall 2023-09-21 15:03:23 +12:00 committed by Andrew Bartlett
parent a2e6df0311
commit b3f92b475c
1 changed files with 22 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
lib/fuzzing/fuzz_sddl_parse.c
View File

@ -18,6 +18,7 @@
#include "includes.h"
#include "libcli/security/security.h"
#include "librpc/gen_ndr/conditional_ace.h"
#include "fuzzing/fuzzing.h"
#define MAX_LENGTH (100 * 1024 - 1)
@ -55,6 +56,27 @@ int LLVMFuzzerTestOneInput(const uint8_t *input, size_t len)
}
result = sddl_encode(mem_ctx, sd1, &dom_sid);
sd2 = sddl_decode(mem_ctx, result, &dom_sid);
if (sd2 == NULL) {
if (strlen(result) > CONDITIONAL_ACE_MAX_LENGTH) {
/*
* This could fail if a unicode string or
* attribute name that contains escapable
* bytes (e.g '\x0b') in an unescaped form in
* the original string ends up with them in
* the escaped form ("%000b") in the result
* string, making the entire attribute name
* too long for the arbitrary limit we set for
* SDDL attribute names.
*
* We could increase that arbitrary limit (to,
* say, CONDITIONAL_ACE_MAX_LENGTH * 5), but
* that is getting very far from real world
* needs.
*/
goto end;
}
abort();
}
ok = security_descriptor_equal(sd1, sd2);
if (!ok) {
abort();