Code cleanup

This commit is contained in:
angt
2016-01-06 17:57:12 +01:00
parent 13dcdc5afb
commit cfbb77c710
3 changed files with 6 additions and 11 deletions

View File

@@ -268,7 +268,7 @@ static char *sk_get_name (int fd)
static struct addrinfo *ai_create (const char *host, const char *port, int listener) static struct addrinfo *ai_create (const char *host, const char *port, int listener)
{ {
if (!port || !port[0]) { if (str_empty(port)) {
gt_log("port is not valid\n"); gt_log("port is not valid\n");
return NULL; return NULL;
} }
@@ -714,7 +714,7 @@ static int gt_setup_secretkey (struct crypto_ctx *ctx, char *keyfile)
{ {
const size_t size = sizeof(ctx->skey); const size_t size = sizeof(ctx->skey);
if (!keyfile) { if (str_empty(keyfile)) {
char buf[2*size+1]; char buf[2*size+1];
randombytes_buf(ctx->skey, size); randombytes_buf(ctx->skey, size);
@@ -1055,7 +1055,7 @@ int main (int argc, char **argv)
char *sockname = sk_get_name(sock.fd); char *sockname = sk_get_name(sock.fd);
if (!sockname) { if (str_empty(sockname)) {
close(sock.fd); close(sock.fd);
continue; continue;
} }

View File

@@ -5,7 +5,7 @@
int option_str (void *data, int argc, char **argv) int option_str (void *data, int argc, char **argv)
{ {
if (argc<2 || !argv[1]) { if (argc<2 || str_empty(argv[1])) {
gt_print("option `%s' need a string argument\n", argv[0]); gt_print("option `%s' need a string argument\n", argv[0]);
return -1; return -1;
} }
@@ -17,7 +17,7 @@ int option_str (void *data, int argc, char **argv)
int option_long (void *data, int argc, char **argv) int option_long (void *data, int argc, char **argv)
{ {
if (argc<2 || !argv[1]) { if (argc<2 || str_empty(argv[1])) {
gt_print("option `%s' need an integer argument\n", argv[0]); gt_print("option `%s' need an integer argument\n", argv[0]);
return -1; return -1;
} }

View File

@@ -44,12 +44,7 @@ static inline size_t str_len (const char *restrict str)
if (!str) if (!str)
return 0; return 0;
size_t i = 0; return strlen(str);
while (str[i])
i++;
return i;
} }
static inline char *str_cat (const char *const strs[], size_t count) static inline char *str_cat (const char *const strs[], size_t count)