Allocation error in CvBoost::update_weights (Patch #1524)


Added by benlemna - about 13 years ago. Updated almost 13 years ago.


Status:Done Start date:
Priority:Normal Due date:
Assignee:Vadim Pisarevsky % Done:

0%

Category:ml
Target version:2.4.0
Affected version: Operating System:
Difficulty: HW Platform:
Pull request:

Description

In boost.cpp line 1129
int _buf_size = (params.boost_type LOGIT) || (params.boost_type GENTLE) ? data->sample_count*sizeof(int) : 0;

needs to be changed to

size_t _buf_size = (params.boost_type LOGIT) || (params.boost_type GENTLE) ? data->sample_count*sizeof(int) : 0;

otherwise boost is unable to train a classifier for a large amount of training points.


Associated revisions

Revision 8bb422b1
Added by Vadim Pisarevsky almost 13 years ago

fixed potential buffer size overflow in Boost::update_weight, bug #1524 (thanks to benlemna for the patch)

History

Updated by Alexander Shishkov almost 13 years ago

  • Tracker changed from Feature to Patch
  • Priority changed from High to Normal
  • Target version deleted ()

Updated by Alexander Shishkov almost 13 years ago

  • Assignee deleted (Maria Dimashova)

Updated by Vadim Pisarevsky almost 13 years ago

thanks! your patch was applied in SVN trunk, r7633

  • Status changed from Open to Done
  • Assignee set to Vadim Pisarevsky

Updated by Alexander Shishkov almost 13 years ago

  • Target version set to 2.4.0

Also available in: Atom PDF