Skip to content

Commit

Permalink
tcp transport: use preallocated SP objects
Browse files Browse the repository at this point in the history
This follows a pattern we started earlier with IPC.
  • Loading branch information
gdamore committed Dec 16, 2024
1 parent 1a552b1 commit 61e3f11
Show file tree
Hide file tree
Showing 2 changed files with 76 additions and 155 deletions.
1 change: 1 addition & 0 deletions src/core/pipe.c
Original file line number Diff line number Diff line change
Expand Up @@ -299,6 +299,7 @@ pipe_create(nni_pipe **pp, nni_sock *sock, nni_sp_tran *tran, nni_dialer *d,
rv2 = tops->p_init(tran_data, p);
rv3 = pops->pipe_init(proto_data, p, sock_data);
if (rv1 != 0 || rv2 != 0 || rv3 != 0) {
nni_panic("HERE");

Check warning on line 302 in src/core/pipe.c

View check run for this annotation

Codecov / codecov/patch

src/core/pipe.c#L302

Added line #L302 was not covered by tests
nni_pipe_close(p);
nni_pipe_rele(p);
return (rv1 ? rv1 : rv2 ? rv2 : rv3);
Expand Down
Loading

0 comments on commit 61e3f11

Please sign in to comment.