diff options
author | Michael Catanzaro <mcatanzaro@gnome.org> | 2020-09-10 14:57:37 -0500 |
---|---|---|
committer | Michael Catanzaro <mcatanzaro@gnome.org> | 2020-09-10 15:02:15 -0500 |
commit | 83cee994952ceb2ff4c818de78f7758c75549e3d (patch) | |
tree | eeb429573e14ff0f6c9d05ba48c5877239206bd5 | |
parent | cd619d3e9d683237f6317f979d5c6a7290d7e429 (diff) | |
download | libproxy-git-83cee994952ceb2ff4c818de78f7758c75549e3d.tar.gz |
Fix mismatched new[]/delete[] in proxy.cpp
Using the wrong delete operator is undefined behavior.
All this manual new/delete really ought to be replaced by
std::unique_ptr, but this will suffice for now.
-rw-r--r-- | libproxy/proxy.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/libproxy/proxy.cpp b/libproxy/proxy.cpp index 2d01d53..72d0039 100644 --- a/libproxy/proxy.cpp +++ b/libproxy/proxy.cpp @@ -244,7 +244,7 @@ void proxy_factory::check_network_topology() { vector<wpad_extension*> wpads = this->mm.get_extensions<wpad_extension>(); for (vector<wpad_extension*>::iterator j=wpads.begin() ; j != wpads.end() ; j++) (*j)->rewind(); - if (this->pac) delete this->pac; + if (this->pac) delete[] this->pac; this->pac = NULL; break; } @@ -313,7 +313,7 @@ bool proxy_factory::expand_wpad(const url &confurl) rtv = true; /* If the config has just changed from PAC to WPAD, clear the PAC */ if (!this->wpad) { - if (this->pac) delete this->pac; + if (this->pac) delete[] this->pac; if (this->pacurl) delete this->pacurl; this->pac = NULL; this->pacurl = NULL; @@ -381,7 +381,7 @@ bool proxy_factory::expand_pac(url &confurl) if (this->pac) { if (this->pacurl->to_string() != confurl.to_string()) { delete this->pacurl; - delete this->pac; + delete[] this->pac; this->pacurl = NULL; this->pac = NULL; } @@ -424,7 +424,7 @@ void proxy_factory::run_pac(url &realurl, const url &confurl, vector<string> &re void proxy_factory::clear_cache() { this->wpad = false; - if (this->pac) { delete this->pac; this->pac = NULL; } + if (this->pac) { delete[] this->pac; this->pac = NULL; } if (this->pacurl) { delete this->pacurl; this->pacurl = NULL; } } |