diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2020-06-24 13:26:05 +0200 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2020-06-25 08:51:21 +0200 |
commit | f23810da209d3fb4bde156a09a1fa93957c82a49 (patch) | |
tree | 353f6b7db81069942c1a8ff32550a7a96f0d5181 /src/fuzz | |
parent | 85fbebe61a1aec2f86e36fb464283b6b55d3d76d (diff) | |
download | systemd-f23810da209d3fb4bde156a09a1fa93957c82a49.tar.gz |
fuzz-udev-rules: -ENOBUFS should be accepted too
In udev_rules_load(), add a debug message. Otherwise there'd be no indication
that parsing was aborted.
Diffstat (limited to 'src/fuzz')
-rw-r--r-- | src/fuzz/fuzz-udev-rules.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/fuzz/fuzz-udev-rules.c b/src/fuzz/fuzz-udev-rules.c index dd3c403ebb..7530d80bef 100644 --- a/src/fuzz/fuzz-udev-rules.c +++ b/src/fuzz/fuzz-udev-rules.c @@ -13,6 +13,7 @@ int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) { _cleanup_(udev_rules_freep) UdevRules *rules = NULL; _cleanup_fclose_ FILE *f = NULL; _cleanup_(unlink_tempfilep) char filename[] = "/tmp/fuzz-udev-rules.XXXXXX"; + int r; if (!getenv("SYSTEMD_LOG_LEVEL")) { log_set_max_level_realm(LOG_REALM_UDEV, LOG_CRIT); @@ -25,7 +26,11 @@ int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) { fflush(f); assert_se(rules = udev_rules_new(RESOLVE_NAME_EARLY)); - assert_se(udev_rules_parse_file(rules, filename) == 0); + r = udev_rules_parse_file(rules, filename); + log_info_errno(r, "Parsing %s: %m", filename); + assert_se(IN_SET(r, + 0, /* OK */ + -ENOBUFS /* line length exceeded */)); return 0; } |