fix for writing null terminated buffers

node 0.5.10+ changed how null terminated buffers are written. The
null terminator is no longer written explicitly.

fixes #63

Author: defunctzombie

lib/writer.js

@@ -15,7 +15,7 @@ p._ensure = function(size) {
   var remaining = this.buffer.length - this.offset;
   if(remaining < size) {
     var oldBuffer = this.buffer;
-    this.buffer = Buffer(oldBuffer.length + size);
+    this.buffer = new Buffer(oldBuffer.length + size);
     oldBuffer.copy(this.buffer);
   }
 }
@@ -40,20 +40,20 @@ p.addCString = function(string) {
   //just write a 0 for empty or null strings
   if(!string) {
     this._ensure(1);
-    this.buffer[this.offset++] = 0;
-    return this;
+  } else {
+    var len = Buffer.byteLength(string);
+    this._ensure(len + 1); //+1 for null terminator
+    this.buffer.write(string, this.offset, len);
+    this.offset += len;
   }
-  var len = Buffer.byteLength(string) + 1;
-  this._ensure(len);
-  this.buffer.write(string, this.offset);
-  this.offset += len;
-  this.buffer[this.offset] = 0; //add null terminator
+
+  this.buffer[this.offset++] = 0; // null terminator
   return this;
 }
 
 p.addChar = function(char) {
   this._ensure(1);
-  this.buffer.write(char, this.offset);
+  this.buffer.write(char, this.offset, 1);
   this.offset++;
   return this;
 }