diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2020-01-11 22:53:34 +0100 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2020-01-12 17:17:44 +0100 |
| commit | ad0aa27a9dfd027ffb62b86dfe8a09feb0fee0d8 (patch) | |
| tree | a7378517025666cdca40ea9500431ca6b971d1fb /src/tool_cb_rea.c | |
| parent | 29babeafec2cbe4a598a4d90d82bb80b7534f959 (diff) | |
curl: remove 'config' field from OutStruct
As it was just unnecessary duplicated information already stored in the
'per_transfer' struct and that's around mostly anyway.
The duplicated pointer caused problems when the code flow was aborted
before the dupe was filled in and could cause a NULL pointer access.
Reported-by: Brian Carpenter
Fixes #4807
Closes #4810
Diffstat (limited to 'src/tool_cb_rea.c')
| -rw-r--r-- | src/tool_cb_rea.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/tool_cb_rea.c b/src/tool_cb_rea.c index 03ed4a467..78a169fb0 100644 --- a/src/tool_cb_rea.c +++ b/src/tool_cb_rea.c @@ -5,7 +5,7 @@ * | (__| |_| | _ <| |___ * \___|\___/|_| \_\_____| * - * Copyright (C) 1998 - 2019, Daniel Stenberg, <daniel@haxx.se>, et al. + * Copyright (C) 1998 - 2020, Daniel Stenberg, <daniel@haxx.se>, et al. * * This software is licensed as described in the file COPYING, which * you should have received as part of this distribution. The terms @@ -63,8 +63,7 @@ int tool_readbusy_cb(void *clientp, curl_off_t ultotal, curl_off_t ulnow) { struct per_transfer *per = clientp; - struct OutStruct *outs = &per->outs; - struct OperationConfig *config = outs->config; + struct OperationConfig *config = per->config; (void)dltotal; /* unused */ (void)dlnow; /* unused */ |