krun: negative return value is used as second argument to crun_make_error()

[eriksjolund]

ret is a negative number here

crun/src/libcrun/handlers/krun.c

Lines 432 to 433 in 617e653

	if (UNLIKELY (ret < 0))
	return ret;

Because of that the second argument passed to crun_make_error() is negative:

crun/src/libcrun/container.c

Line 1701 in 617e653

return crun_make_error (err, ret, "exec container process failed with handler as `%s`", entrypoint_args->custom_handler->vtable->name);

The function crun_make_error() expects the second argument to be non-negative.

[eriksjolund]

See also: #1954 (comment)

[giuseppe]

opened a PR: #1959