diff options
| author | Simon Glass <sjg@chromium.org> | 2020-02-03 07:36:02 -0700 | 
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2020-02-05 19:33:46 -0700 | 
| commit | 0db1b4305ad5ba8ef9c41d2f07697c77a04787e2 (patch) | |
| tree | eb3d8985ed675c8ee220d4b001c43215575b44f2 | |
| parent | f72bdc60b230f5e5d77eaad0b2defe07a46c02a9 (diff) | |
sandbox: Drop use of special os_malloc() where possible
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>
| -rw-r--r-- | arch/sandbox/cpu/eth-raw-os.c | 6 | ||||
| -rw-r--r-- | arch/sandbox/cpu/os.c | 24 | 
2 files changed, 15 insertions, 15 deletions
| diff --git a/arch/sandbox/cpu/eth-raw-os.c b/arch/sandbox/cpu/eth-raw-os.c index 8d05bc2eda0..da01d1addf9 100644 --- a/arch/sandbox/cpu/eth-raw-os.c +++ b/arch/sandbox/cpu/eth-raw-os.c @@ -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; diff --git a/arch/sandbox/cpu/os.c b/arch/sandbox/cpu/os.c index 79094fb7f35..d5e5b561b6a 100644 --- a/arch/sandbox/cpu/os.c +++ b/arch/sandbox/cpu/os.c @@ -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); | 
