Its only failure mode was name == NULL, but we only ever pass in
optarg from a required-argument flag. Drop the pretense and convert
that conditional to an assertion, because it's actually actively
harmful: GCC 14, at least, will deduce that the error path means that
optarg can be NULL and raise a warning about passing NULL to an %s
specifier.

PR: 298192