Skip to content

Commit c75eaf5

Browse files
authored
assert return of mcd_rpc_message_to_iovecs (#1360)
* assert return of `mcd_rpc_message_to_iovecs` Current implementation is expected not to return `NULL`. `BSON_ASSERT` is added for consistency with other calls and to future-proof. * assert return of `mcd_rpc_message_to_iovecs` in mock server
1 parent 9affd70 commit c75eaf5

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

src/libmongoc/src/mongoc/mongoc-cluster.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3580,6 +3580,7 @@ _mongoc_cluster_run_opmsg_send (mongoc_cluster_t *cluster,
35803580

35813581
size_t num_iovecs = 0u;
35823582
mongoc_iovec_t *const iovecs = mcd_rpc_message_to_iovecs (rpc, &num_iovecs);
3583+
BSON_ASSERT (iovecs);
35833584

35843585
mcd_rpc_message_egress (rpc);
35853586
const bool res = _mongoc_stream_writev_full (server_stream->stream,

src/libmongoc/tests/mock_server/mock-server.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2135,6 +2135,7 @@ _mock_server_reply_with_stream (mock_server_t *server,
21352135

21362136
size_t iovcnt;
21372137
mongoc_iovec_t *const iov = mcd_rpc_message_to_iovecs (rpc, &iovcnt);
2138+
BSON_ASSERT (iov);
21382139

21392140
size_t expected = 0;
21402141
for (size_t i = 0u; i < iovcnt; i++) {

0 commit comments

Comments
 (0)