diff options
author | Dan McGee <dpmcgee@gmail.com> | 2011-11-20 16:48:16 -0600 |
---|---|---|
committer | Dan McGee <dpmcgee@gmail.com> | 2011-11-20 16:48:16 -0600 |
commit | 36204b1a6e611f44890b5a5d8faa303f3ace251d (patch) | |
tree | 91d7392850c86c7e37262af1659f1f31c2111227 | |
parent | a17dd711cbe5b2b2742faefaf710414e21175f12 (diff) | |
download | onkyocontrol-36204b1a6e611f44890b5a5d8faa303f3ace251d.tar.gz onkyocontrol-36204b1a6e611f44890b5a5d8faa303f3ace251d.zip |
Small code cleanups
Signed-off-by: Dan McGee <dpmcgee@gmail.com>
-rw-r--r-- | command.c | 2 | ||||
-rw-r--r-- | onkyo.c | 2 | ||||
-rw-r--r-- | receiver.c | 4 | ||||
-rw-r--r-- | util.c | 2 |
4 files changed, 8 insertions, 2 deletions
@@ -78,6 +78,8 @@ static int cmd_attempt(struct receiver *rcvr, sprintf(fullcmd, "%s%s", cmd->prefix, arg); q = malloc(sizeof(struct cmdqueue)); + if(!q) + return -1; q->hash = hash_sdbm(fullcmd); q->cmd = fullcmd; q->next = NULL; @@ -115,6 +115,8 @@ static int open_connection(int fd) if(!ptr) { ptr = calloc(1, sizeof(struct conn)); + if(!ptr) + return -1; } if(!ptr->recv_buf) { ptr->recv_buf = calloc(BUF_SIZE, sizeof(char)); @@ -82,10 +82,12 @@ static char *next_rcvr_command(struct receiver *rcvr) */ int rcvr_send_command(struct receiver *rcvr) { + char *cmd; + if(!rcvr->queue) return -1; - char *cmd = next_rcvr_command(rcvr); + cmd = next_rcvr_command(rcvr); if(cmd) { ssize_t retval; size_t cmdsize = strlen(START_SEND) + strlen(cmd) @@ -80,9 +80,9 @@ ssize_t xwrite(int fd, const void *buf, size_t len) unsigned long hash_sdbm(const char *str) { unsigned long hash = 0; + int c; if(!str) return hash; - int c; while((c = *str++)) hash = c + (hash << 6) + (hash << 16) - hash; |