]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
sandbox: Drop use of special os_malloc() where possible
authorSimon Glass <sjg@chromium.org>
Mon, 3 Feb 2020 14:36:02 +0000 (07:36 -0700)
committerSimon Glass <sjg@chromium.org>
Thu, 6 Feb 2020 02:33:46 +0000 (19:33 -0700)
Some sandbox files are not built with U-Boot headers, so with the renamed
malloc functions there is now no need to use the special os_... allocation
functions to access the system routines. Instead we can just call them
directly.

Update the affected files accordingly.

Signed-off-by: Simon Glass <sjg@chromium.org>
arch/sandbox/cpu/eth-raw-os.c
arch/sandbox/cpu/os.c

index 8d05bc2eda039a21598950075bc9bb0ae74bffe1..da01d1addf9c2af3026589e41d78b10e08fad621 100644 (file)
@@ -74,7 +74,7 @@ static int _raw_packet_start(struct eth_sandbox_raw_priv *priv,
 
        /* Prepare device struct */
        priv->local_bind_sd = -1;
-       priv->device = os_malloc(sizeof(struct sockaddr_ll));
+       priv->device = malloc(sizeof(struct sockaddr_ll));
        if (priv->device == NULL)
                return -ENOMEM;
        device = priv->device;
@@ -147,7 +147,7 @@ static int _local_inet_start(struct eth_sandbox_raw_priv *priv)
        /* Prepare device struct */
        priv->local_bind_sd = -1;
        priv->local_bind_udp_port = 0;
-       priv->device = os_malloc(sizeof(struct sockaddr_in));
+       priv->device = malloc(sizeof(struct sockaddr_in));
        if (priv->device == NULL)
                return -ENOMEM;
        device = priv->device;
@@ -282,7 +282,7 @@ int sandbox_eth_raw_os_recv(void *packet, int *length,
 
 void sandbox_eth_raw_os_stop(struct eth_sandbox_raw_priv *priv)
 {
-       os_free(priv->device);
+       free(priv->device);
        priv->device = NULL;
        close(priv->sd);
        priv->sd = -1;
index 79094fb7f353b4981d9f446c8c0eb6d34d4c3ce0..d5e5b561b6a49eeaf172a82c78bf626d9b98e775 100644 (file)
@@ -137,7 +137,7 @@ int os_read_file(const char *fname, void **bufp, int *sizep)
                printf("Cannot seek to start of file '%s'\n", fname);
                goto err;
        }
-       *bufp = os_malloc(size);
+       *bufp = malloc(size);
        if (!*bufp) {
                printf("Not enough memory to read file '%s'\n", fname);
                ret = -ENOMEM;
@@ -306,8 +306,8 @@ int os_parse_args(struct sandbox_state *state, int argc, char *argv[])
        state->argv = argv;
 
        /* dynamically construct the arguments to the system getopt_long */
-       short_opts = os_malloc(sizeof(*short_opts) * num_options * 2 + 1);
-       long_opts = os_malloc(sizeof(*long_opts) * num_options);
+       short_opts = malloc(sizeof(*short_opts) * num_options * 2 + 1);
+       long_opts = malloc(sizeof(*long_opts) * num_options);
        if (!short_opts || !long_opts)
                return 1;
 
@@ -385,7 +385,7 @@ void os_dirent_free(struct os_dirent_node *node)
 
        while (node) {
                next = node->next;
-               os_free(node);
+               free(node);
                node = next;
        }
 }
@@ -410,7 +410,7 @@ int os_dirent_ls(const char *dirname, struct os_dirent_node **headp)
        /* Create a buffer upfront, with typically sufficient size */
        dirlen = strlen(dirname) + 2;
        len = dirlen + 256;
-       fname = os_malloc(len);
+       fname = malloc(len);
        if (!fname) {
                ret = -ENOMEM;
                goto done;
@@ -423,7 +423,7 @@ int os_dirent_ls(const char *dirname, struct os_dirent_node **headp)
                        ret = errno;
                        break;
                }
-               next = os_malloc(sizeof(*node) + strlen(entry->d_name) + 1);
+               next = malloc(sizeof(*node) + strlen(entry->d_name) + 1);
                if (!next) {
                        os_dirent_free(head);
                        ret = -ENOMEM;
@@ -432,10 +432,10 @@ int os_dirent_ls(const char *dirname, struct os_dirent_node **headp)
                if (dirlen + strlen(entry->d_name) > len) {
                        len = dirlen + strlen(entry->d_name);
                        old_fname = fname;
-                       fname = os_realloc(fname, len);
+                       fname = realloc(fname, len);
                        if (!fname) {
-                               os_free(old_fname);
-                               os_free(next);
+                               free(old_fname);
+                               free(next);
                                os_dirent_free(head);
                                ret = -ENOMEM;
                                goto done;
@@ -469,7 +469,7 @@ int os_dirent_ls(const char *dirname, struct os_dirent_node **headp)
 
 done:
        closedir(dir);
-       os_free(fname);
+       free(fname);
        return ret;
 }
 
@@ -586,7 +586,7 @@ static int add_args(char ***argvp, char *add_args[], int count)
        for (argc = 0; (*argvp)[argc]; argc++)
                ;
 
-       argv = os_malloc((argc + count + 1) * sizeof(char *));
+       argv = malloc((argc + count + 1) * sizeof(char *));
        if (!argv) {
                printf("Out of memory for %d argv\n", count);
                return -ENOMEM;
@@ -669,7 +669,7 @@ static int os_jump_to_file(const char *fname)
                os_exit(2);
 
        err = execv(fname, argv);
-       os_free(argv);
+       free(argv);
        if (err) {
                perror("Unable to run image");
                printf("Image filename '%s'\n", fname);