Merge pull request #926 from zbjornson/fix-mem-leak

Fix memory leak in Canvas constructor

Author: LinusU

src/Canvas.cc

@@ -617,7 +617,6 @@ NAN_METHOD(Canvas::RegisterFont) {
 
 Canvas::Canvas(Backend* backend) : ObjectWrap() {
   _backend = backend;
-  this->backend()->createSurface();
 }
 
 /*

src/backend/ImageBackend.cc

@@ -17,9 +17,10 @@ ImageBackend::~ImageBackend()
 
 cairo_surface_t* ImageBackend::createSurface()
 {
+	assert(!this->surface);
 	this->surface = cairo_image_surface_create(CAIRO_FORMAT_ARGB32, width, height);
 	assert(this->surface);
-  Nan::AdjustExternalMemory(4 * width * height);
+	Nan::AdjustExternalMemory(4 * width * height);
 
 	return this->surface;
 }
@@ -35,7 +36,6 @@ cairo_surface_t* ImageBackend::recreateSurface()
 	return createSurface();
 }
 
-
 Nan::Persistent<FunctionTemplate> ImageBackend::constructor;
 
 void ImageBackend::Initialize(Handle<Object> target)

src/backend/PdfBackend.cc

@@ -13,6 +13,8 @@ using namespace v8;
 PdfBackend::PdfBackend(int width, int height)
 	: Backend("pdf", width, height)
 {
+	_closure = malloc(sizeof(closure_t));
+	assert(_closure);
 	createSurface();
 }
 
@@ -28,8 +30,6 @@ PdfBackend::~PdfBackend()
 
 cairo_surface_t* PdfBackend::createSurface()
 {
-	_closure = malloc(sizeof(closure_t));
-	assert(_closure);
 	cairo_status_t status = closure_init((closure_t*)_closure, this->canvas, 0, PNG_NO_FILTERS);
 	assert(status == CAIRO_STATUS_SUCCESS);
 

src/backend/SvgBackend.cc

@@ -15,7 +15,6 @@ SvgBackend::SvgBackend(int width, int height)
 {
 	_closure = malloc(sizeof(closure_t));
 	assert(_closure);
-
 	createSurface();
 }