Assert that blank malloc and calloc are allocated and remove error checking
authorJonathan Dieter <jdieter@gmail.com>
Fri, 14 Sep 2018 13:45:04 +0000 (14:45 +0100)
committerJonathan Dieter <jdieter@gmail.com>
Fri, 14 Sep 2018 13:45:04 +0000 (14:45 +0100)
Signed-off-by: Jonathan Dieter <jdieter@gmail.com>
src/lib/buzhash/buzhash.c
src/unzck.c
src/zck.c

index f3ed195edb930f3fbc4f13bf6f0d0a4b33775a77..780febd4c7fbc9ba7646d79e921ee87f3b76e527 100644 (file)
@@ -23,6 +23,7 @@
 
 #include "buzhash.h"
 
+#include <assert.h>
 #include <string.h>
 
 static uint32_t rol32(uint32_t v, uint32_t s) {
@@ -104,6 +105,7 @@ uint32_t buzhash_update (buzHash *b, const char *s, size_t window) {
         if(b->window)
             free(b->window);
         b->window = calloc(1, window);
+        assert(b->window);
         b->window_loc = 0;
         b->window_fill = 0;
         b->window_size = window;
index a848c5dabc4a4fdcf426ed9f3a452ec652a611e7..fdded0e42e6201e1e957929b9830ce2a77f9ef67 100644 (file)
@@ -26,6 +26,7 @@
 
 #define _GNU_SOURCE
 
+#include <assert.h>
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
@@ -116,6 +117,7 @@ int main (int argc, char *argv[]) {
     }
     char *base_name = basename(arguments.args[0]);
     char *out_name = malloc(strlen(base_name) - 3);
+    assert(out_name);
     snprintf(out_name, strlen(base_name) - 3, "%s", base_name);
 
     int dst_fd = STDOUT_FILENO;
@@ -132,13 +134,8 @@ int main (int argc, char *argv[]) {
     bool good_exit = false;
 
     zckCtx *zck = zck_create();
-    if(zck == NULL) {
-        dprintf(STDERR_FILENO, "%s", zck_get_error(NULL));
-        zck_clear_error(NULL);
-        goto error1;
-    }
-
     char *data = malloc(BUF_SIZE);
+    assert(data);
     if(!zck_init_read(zck, src_fd))
         goto error2;
 
@@ -172,7 +169,6 @@ error2:
     if(!good_exit)
         dprintf(STDERR_FILENO, "%s", zck_get_error(zck));
     zck_free(&zck);
-error1:
     if(!good_exit)
         unlink(out_name);
     free(out_name);
index 5fd7fc0ffaadf764be276014b8af082900e1a222..26d7996dc70f94ebdd4a9aa2c1f53ced68d8d85f 100644 (file)
--- a/src/zck.c
+++ b/src/zck.c
@@ -26,6 +26,7 @@
 
 #define _GNU_SOURCE
 
+#include <assert.h>
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
@@ -132,10 +133,12 @@ int main (int argc, char *argv[]) {
     if(arguments.output == NULL) {
         base_name = basename(arguments.args[0]);
         out_name = malloc(strlen(base_name) + 5);
+        assert(out_name);
         snprintf(out_name, strlen(base_name) + 5, "%s.zck", base_name);
     } else {
         base_name = arguments.output;
         out_name = malloc(strlen(base_name) + 1);
+        assert(out_name);
         snprintf(out_name, strlen(base_name) + 1, "%s", base_name);
     }
 
@@ -160,6 +163,7 @@ int main (int argc, char *argv[]) {
             exit(1);
         }
         dict = malloc(dict_size);
+        assert(dict);
         if(read(dict_fd, dict, dict_size) < dict_size) {
             free(dict);
             perror("Error reading dict:");
@@ -231,6 +235,7 @@ int main (int argc, char *argv[]) {
     }
     if(in_size > 0) {
         data = malloc(in_size);
+        assert(data);
         if(read(in_fd, data, in_size) < in_size) {
             dprintf(STDERR_FILENO, "Unable to read from input file\n");
             exit(1);