Skip to content

Commit a0d9aae

Browse files
committed
broker: fix cert_create() error handling
Problem: in overlay_create(), all cert_create() failures are reported as a generic ENOMEM, without calling out cert_create(). Since cert_create() sets errno, allow errno to propagate on failure. Also add a log message indicating that cert_create() is the cause of the failure since overlay_create() is a long function. This problem was noted in #6151.
1 parent 966cae5 commit a0d9aae

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/broker/overlay.c

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2493,8 +2493,10 @@ struct overlay *overlay_create (flux_t *h,
24932493
goto error;
24942494
if (flux_msg_handler_addvec (h, htab, ov, &ov->handlers) < 0)
24952495
goto error;
2496-
if (!(ov->cert = cert_create ()))
2497-
goto nomem;
2496+
if (!(ov->cert = cert_create ())) {
2497+
log_err ("could not create curve certificate");
2498+
goto error;
2499+
}
24982500
if (!(ov->health_requests = flux_msglist_create ())
24992501
|| !(ov->trace_requests = flux_msglist_create ()))
25002502
goto error;

0 commit comments

Comments
 (0)