diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2014-03-08 22:58:37 +0100 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2014-03-10 12:24:10 +0100 |
commit | 341055ddb3bfaf3297aa068abf0d2412bc4cfd0b (patch) | |
tree | 6ffca684e0cae4d6c74df34b566970122bf074e4 /daemons/clvmd/clvmd.c | |
parent | c44ede31a39598c30634ff7b20e45e4600df5d37 (diff) | |
download | lvm2-341055ddb3bfaf3297aa068abf0d2412bc4cfd0b.tar.gz |
cleanup: clvmd use struct initializers
Simplify code, and use compiler capability to
initilize struct members on its own.
Diffstat (limited to 'daemons/clvmd/clvmd.c')
-rw-r--r-- | daemons/clvmd/clvmd.c | 24 |
1 files changed, 8 insertions, 16 deletions
diff --git a/daemons/clvmd/clvmd.c b/daemons/clvmd/clvmd.c index 43fecb659..1fc369447 100644 --- a/daemons/clvmd/clvmd.c +++ b/daemons/clvmd/clvmd.c @@ -1121,15 +1121,13 @@ static int verify_message(char *buf, int len) static void dump_message(char *buf, int len) { - unsigned char row[8]; + unsigned char row[8] = { 0 }; char str[9]; int i, j, pos = 0; if (len > 128) len = 128; - memset(row, 0, sizeof(row)); - for (i = 0; i < len; i++) { row[pos++] = buf[i]; @@ -1247,9 +1245,7 @@ static int read_from_local_sock(struct local_client *thisfd) int len; int argslen; int missing_len; - char buffer[PIPE_BUF + 1]; - - memset(buffer, 0, sizeof(buffer)); + char buffer[PIPE_BUF + 1] = { 0 }; len = read(thisfd->fd, buffer, sizeof(buffer) - 1); if (len == -1 && errno == EINTR) @@ -1982,7 +1978,11 @@ static void send_version_message(void) { char message[sizeof(struct clvm_header) + sizeof(int) * 3]; struct clvm_header *msg = (struct clvm_header *) message; - int version_nums[3]; + int version_nums[3] = { + htonl(CLVMD_MAJOR_VERSION), + htonl(CLVMD_MINOR_VERSION), + htonl(CLVMD_PATCH_VERSION) + }; msg->cmd = CLVMD_CMD_VERSION; msg->status = 0; @@ -1990,10 +1990,6 @@ static void send_version_message(void) msg->clientid = 0; msg->arglen = sizeof(version_nums); - version_nums[0] = htonl(CLVMD_MAJOR_VERSION); - version_nums[1] = htonl(CLVMD_MINOR_VERSION); - version_nums[2] = htonl(CLVMD_PATCH_VERSION); - memcpy(&msg->args, version_nums, sizeof(version_nums)); hton_clvm(msg); @@ -2379,11 +2375,7 @@ static if_type_t get_cluster_type(void) size_t namelen = sizeof(buf); hdb_handle_t cluster_handle; hdb_handle_t clvmd_handle; - confdb_callbacks_t callbacks = { - .confdb_key_change_notify_fn = NULL, - .confdb_object_create_change_notify_fn = NULL, - .confdb_object_delete_change_notify_fn = NULL - }; + confdb_callbacks_t callbacks = { 0 }; result = confdb_initialize (&handle, &callbacks); if (result != CS_OK) |