]> git.ipfire.org Git - thirdparty/systemd.git/commit
coredump: get rid of a bogus assertion
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Mon, 26 May 2025 13:24:04 +0000 (15:24 +0200)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Wed, 28 May 2025 22:10:55 +0000 (00:10 +0200)
commit13902e025321242b1d95c6d8b4e482b37f58cdef
tree3881afd4d9753ae06cd7be2f5ffe7d21165a44cd
parent8fc7b2a211eb13ef1a94250b28e1c79cab8bdcb9
coredump: get rid of a bogus assertion

The check looks plausible, but when I started checking whether it needs
to be lowered for the recent changes, I realized that it doesn't make
much sense.

context_parse_iovw() is called from a few places, e.g.:
- process_socket(), where the other side controls the contents of the
  message. We already do other checks on the correctness of the message
  and this assert is not needed.
- gather_pid_metadata_from_argv(), which is called after
  inserting MESSAGE_ID= and PRIORITY= into the array, so there is no
  direct relation between _META_ARGV_MAX and the number of args in the
  iovw.
- gather_pid_metadata_from_procfs(), where we insert a bazillion fields,
  but without any relation to _META_ARGV_MAX.

Since we already separately check if the required stuff was set, drop this
misleading check.
src/coredump/coredump.c