Fix a bad free space check in icvGrowSeq.

A difference of two pointers was casted to unsigned which can lead to
overflow on 64-bit systems.
pull/3966/head
Boris Daskalov 10 years ago committed by Dikay900
parent 2c73e49dac
commit 3b5530fb67
  1. 2
      modules/core/src/datastructs.cpp

@ -651,7 +651,7 @@ icvGrowSeq( CvSeq *seq, int in_front_of )
/* If there is a free space just after last allocated block
and it is big enough then enlarge the last block.
This can happen only if the new block is added to the end of sequence: */
if( (unsigned)(ICV_FREE_PTR(storage) - seq->block_max) < CV_STRUCT_ALIGN &&
if( (size_t)(ICV_FREE_PTR(storage) - seq->block_max) < CV_STRUCT_ALIGN &&
storage->free_space >= seq->elem_size && !in_front_of )
{
int delta = storage->free_space / elem_size;

Loading…
Cancel
Save