From 18587d132b4d600bff52b1428b78a0667537c9ec Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Beno=C3=AEt=20Blanchon?= <benoit.blanchon@gmail.com>
Date: Thu, 26 Jun 2014 13:14:09 +0200
Subject: [PATCH] Added a test with a NULL string

---
 JsonGeneratorTests/StringBuilder.cpp            | 11 ++++++++++-
 JsonGeneratorTests/StringBuilderAppendTests.cpp |  6 ++++++
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/JsonGeneratorTests/StringBuilder.cpp b/JsonGeneratorTests/StringBuilder.cpp
index ff67d5a9..c7230199 100644
--- a/JsonGeneratorTests/StringBuilder.cpp
+++ b/JsonGeneratorTests/StringBuilder.cpp
@@ -8,6 +8,11 @@
 
 void StringBuilder::append(const char* s)
 {
+    if (!s)
+    {
+        return append("null");        
+    }
+
     char* tail = buffer + length;
 
     while (*s && length<capacity)
@@ -20,7 +25,11 @@ void StringBuilder::append(const char* s)
 
 void StringBuilder::appendEscaped(const char* s)
 {
-    if (length > capacity - 3) return;
+    if (length > capacity - 2)
+    {
+        // not enough from for quotes
+        return;
+    }
 
     buffer[length++] = '"';
 
diff --git a/JsonGeneratorTests/StringBuilderAppendTests.cpp b/JsonGeneratorTests/StringBuilderAppendTests.cpp
index d5350a41..900895d0 100644
--- a/JsonGeneratorTests/StringBuilderAppendTests.cpp
+++ b/JsonGeneratorTests/StringBuilderAppendTests.cpp
@@ -28,6 +28,12 @@ namespace JsonGeneratorTests
             assertResultIs("");
         }
 
+        TEST_METHOD(Null)
+        {
+            append(NULL);
+            assertResultIs("null");
+        }
+
         TEST_METHOD(OneString)
         {
             append("ABCD");