diff options
| author | Sander Vrijders <sander.vrijders@intec.ugent.be> | 2016-02-26 16:08:09 +0100 | 
|---|---|---|
| committer | Sander Vrijders <sander.vrijders@intec.ugent.be> | 2016-02-26 16:08:09 +0100 | 
| commit | f02e1f68905a563e1d73a298e0de0e5da8b2eb9f (patch) | |
| tree | f87e7d2a7e5a9c789a47561b8a715152aac24e43 /src/lib | |
| parent | 0ab26ce5ae6c80313e55e3251cc0ee180419f696 (diff) | |
| download | ouroboros-f02e1f68905a563e1d73a298e0de0e5da8b2eb9f.tar.gz ouroboros-f02e1f68905a563e1d73a298e0de0e5da8b2eb9f.zip | |
lib: Fix two errors
Fixes two errors in the library. The return code of a write to a
socket was ignored. A non literal string was passed to sprintf as a
literal one.
Diffstat (limited to 'src/lib')
| -rw-r--r-- | src/lib/irm.c | 10 | ||||
| -rw-r--r-- | src/lib/sockets.c | 2 | 
2 files changed, 8 insertions, 4 deletions
| diff --git a/src/lib/irm.c b/src/lib/irm.c index 2c9b530c..69a98039 100644 --- a/src/lib/irm.c +++ b/src/lib/irm.c @@ -46,13 +46,17 @@ int irm_create_ipcp(rina_name_t name,          msg.msgs.create_ipcp.ipcp_type = ipcp_type;          buf = serialize_irm_msg(&msg); -        if (buf == NULL) +        if (buf == NULL) { +                close(sockfd);                  return -1; +        } -        write(sockfd, buf->data, buf->size); +        if (write(sockfd, buf->data, buf->size) == -1) { +                close(sockfd); +                return -1; +        }          close(sockfd); -          return 0;  } diff --git a/src/lib/sockets.c b/src/lib/sockets.c index 095c9e5c..0ebad842 100644 --- a/src/lib/sockets.c +++ b/src/lib/sockets.c @@ -45,7 +45,7 @@ int client_socket_open(char * file_name)          }          serv_addr.sun_family = AF_UNIX; -        sprintf(serv_addr.sun_path, file_name); +        sprintf(serv_addr.sun_path, "%s", file_name);          if (connect(sockfd,                      (struct sockaddr *) &serv_addr, | 
