Skip to content

Commit cf00994

Browse files
committed
Make sure the received string is NULL-terminated
Make sure the received string is NULL-terminated. It fixes the Coverity issue no. 460614. Signed-off-by: Lukasz Dorau <[email protected]>
1 parent dec0d62 commit cf00994

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

examples/basic/ipc_shm_ipcapi_producer.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ int main(int argc, char *argv[]) {
180180
memset(recv_buffer, 0, sizeof(recv_buffer));
181181

182182
// receive the consumer's response
183-
len = recv(producer_socket, recv_buffer, sizeof(recv_buffer), 0);
183+
len = recv(producer_socket, recv_buffer, sizeof(recv_buffer) - 1, 0);
184184
if (len < 0) {
185185
fprintf(
186186
stderr,

test/ipc_os_prov_producer.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -205,8 +205,8 @@ int main(int argc, char *argv[]) {
205205
memset(consumer_message, 0, sizeof(consumer_message));
206206

207207
// receive the consumer's response
208-
if (recv(producer_socket, consumer_message, sizeof(consumer_message), 0) <
209-
0) {
208+
if (recv(producer_socket, consumer_message, sizeof(consumer_message) - 1,
209+
0) < 0) {
210210
fprintf(
211211
stderr,
212212
"[producer] ERROR: error while receiving the consumer's message\n");

0 commit comments

Comments
 (0)