diff options
| author | Daniel Gustafsson <daniel@yesql.se> | 2018-04-14 22:42:04 +0200 |
|---|---|---|
| committer | Jay Satiro <raysatiro@yahoo.com> | 2018-04-15 03:00:37 -0400 |
| commit | 94400f32e9016d2eaea2db583f6e213c36b1eb1d (patch) | |
| tree | d8c79e652dfa3a9b52b0056cdbe00a66411a502b /docs/examples/fopen.c | |
| parent | 2b126cd7083ddf1308ebc447cabd1983b16a99fa (diff) | |
| download | gnurl-94400f32e9016d2eaea2db583f6e213c36b1eb1d.tar.gz gnurl-94400f32e9016d2eaea2db583f6e213c36b1eb1d.tar.bz2 gnurl-94400f32e9016d2eaea2db583f6e213c36b1eb1d.zip | |
all: Refactor malloc+memset to use calloc
When a zeroed out allocation is required, use calloc() rather than
malloc() followed by an explicit memset(). The result will be the
same, but using calloc() everywhere increases consistency in the
codebase and avoids the risk of subtle bugs when code is injected
between malloc and memset by accident.
Closes https://github.com/curl/curl/pull/2497
Diffstat (limited to 'docs/examples/fopen.c')
| -rw-r--r-- | docs/examples/fopen.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/docs/examples/fopen.c b/docs/examples/fopen.c index eb27c6ae9..f1706fbe6 100644 --- a/docs/examples/fopen.c +++ b/docs/examples/fopen.c @@ -237,12 +237,10 @@ URL_FILE *url_fopen(const char *url, const char *operation) URL_FILE *file; (void)operation; - file = malloc(sizeof(URL_FILE)); + file = calloc(1, sizeof(URL_FILE)); if(!file) return NULL; - memset(file, 0, sizeof(URL_FILE)); - file->handle.file = fopen(url, operation); if(file->handle.file) file->type = CFTYPE_FILE; /* marked as URL */ |