diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2015-12-09 21:35:49 -0500 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2015-12-13 14:54:47 -0500 |
commit | e64c53fd05496fd9683db8f5130826ad1c6c515d (patch) | |
tree | ee8a88d021e5de87ff14dae5c5116fbeff43b7c1 /src/journal | |
parent | c9811ac800df461b010909f343a2e0251b6b57d4 (diff) |
journal: properly handle an unexpectedly missing field
parse_field() checks if the field has the expected format, and returns
0 if it doesn't. In that case, value and size are not
set. Nevertheless, we would try to continue, and hit an assert in
safe_atou64. This case shouldn't happen, unless sd_j_get_data is borked,
so cleanly assert that we got the expected field.
Also, oom is the only way that parse_field can fail, which we log
already. Instead of outputting a debug statement and carrying on,
treat oom as fatal.
Diffstat (limited to 'src/journal')
0 files changed, 0 insertions, 0 deletions