diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2020-07-07 11:31:17 +0200 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2020-07-07 14:02:16 +0200 |
| commit | dea7f5cc8751ab36c75acb7e3d181edef5e73876 (patch) | |
| tree | 329f7206f5b0ec43bfc9c3f0b8e7d8184e0683d5 /test/TEST-44-LOG-NAMESPACE | |
| parent | xdg-autostart: avoid quadratic behaviour in strv parsing (diff) | |
| download | systemd-dea7f5cc8751ab36c75acb7e3d181edef5e73876.tar.xz systemd-dea7f5cc8751ab36c75acb7e3d181edef5e73876.zip | |
xdg-autostart: ignore all empty entries in multi-string entries
The desktop file specification allows entries like ";;;;;;", full of empty strings.
But looking at the actual list of supported keys [1], empty entries are meaningless
(unless we would allow e.g. the desktop name to be the empty string. But that doesn't
seem very useful either). So let's just simplify our life and skip any empty substrings
entirely.
This would also resolve the fuzzer case:
$ valgrind build/fuzz-xdg-desktop test/fuzz/fuzz-xdg-desktop/oss-fuzz-22812
test/fuzz/fuzz-xdg-desktop/oss-fuzz-22812... ok
==2899241== HEAP SUMMARY:
==2899241== in use at exit: 0 bytes in 0 blocks
==2899241== total heap usage: 484,385 allocs, 484,385 frees, 12,411,330 bytes allocated
↓
==2899650== HEAP SUMMARY:
==2899650== in use at exit: 0 bytes in 0 blocks
==2899650== total heap usage: 1,325 allocs, 1,325 frees, 1,463,602 bytes allocated
Diffstat (limited to 'test/TEST-44-LOG-NAMESPACE')
0 files changed, 0 insertions, 0 deletions