diff options
author | Jay Berkenbilt <ejb@ql.org> | 2018-08-12 18:58:39 +0200 |
---|---|---|
committer | Jay Berkenbilt <ejb@ql.org> | 2018-08-12 18:58:39 +0200 |
commit | b6e414b10b3ae7b28ad16da2027106ec59a99a0a (patch) | |
tree | cecd3bb38d43af6d4fa687414d2565e71c0ac206 /libqpdf/Pl_Flate.cc | |
parent | 4a4736c6954ab17d923a6d2968f34a33e09d714f (diff) | |
download | qpdf-b6e414b10b3ae7b28ad16da2027106ec59a99a0a.tar.zst |
Remove some extraneous null pointer checks (fixes #234)
There were a few places in the code that were checking that a pointer
wasn't null before deleting it, even though C++ has always allowed
delete 0. Most of the code did not perform these checks.
Diffstat (limited to 'libqpdf/Pl_Flate.cc')
-rw-r--r-- | libqpdf/Pl_Flate.cc | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/libqpdf/Pl_Flate.cc b/libqpdf/Pl_Flate.cc index 3becc135..4cd48046 100644 --- a/libqpdf/Pl_Flate.cc +++ b/libqpdf/Pl_Flate.cc @@ -31,11 +31,8 @@ Pl_Flate::Pl_Flate(char const* identifier, Pipeline* next, Pl_Flate::~Pl_Flate() { - if (this->outbuf) - { - delete [] this->outbuf; - this->outbuf = 0; - } + delete [] this->outbuf; + this->outbuf = 0; if (this->initialized) { |