search

Gregwar / mongoose-cpp

C++ binding for Mongoose
Other
101 stars 45 forks source link

Memory leak #24

Open d3roch4 opened 9 years ago

d3roch4 commented 9 years ago

Memory leak in RequestHandler.h file: 29 this allocating a Response and Controller.cpp file: allocating another 113 this Response. At the end apens one will be deleted causing a memory leak.

My suggestion is to move the Controller method :: serverInternalError the Controller class to RequestHandler class, I'll implentar this and commit on my fork.

Gregwar commented 9 years ago

Would you mind making a pull request?

d3roch4 commented 9 years ago

pull request: https://github.com/Gregwar/mongoose-cpp/pull/23