Skip to content

Commit d2b456b

Browse files
committed
Merge pull request #113 from Vikcoc/theor-mem-leak
* Vikcoc/theor-mem-leak: ZIP.cpp move check between asignment and first use
2 parents 1c25971 + eb5649e commit d2b456b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/lib/io/ZIP.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -520,9 +520,9 @@ Find_And_Read_Central_Header()
520520
unsigned int read_size_before_comment=22;
521521
std::streamoff read_start=max_comment_size+read_size_before_comment;
522522
if(read_start>end_position) read_start=end_position;
523+
if(read_start<=0){std::cerr<<"ZIP: Invalid read buffer size"<<std::endl;return false;}
523524
istream.seekg(end_position-read_start);
524525
char *buf=new char[static_cast<unsigned int>(read_start)];
525-
if(read_start<=0){std::cerr<<"ZIP: Invalid read buffer size"<<std::endl;return false;}
526526
istream.read(buf,read_start);
527527
int found=-1;
528528
for(unsigned int i=0;i<read_start-3;i++){

0 commit comments

Comments
 (0)