Disallow control characters in strings

1 files changed, 14 insertions, 0 deletions

diff --git a/ccl_test.go b/ccl_test.go
index e377306..e7405f8 100644
--- a/ccl_test.go
+++ b/ccl_test.go
@@ -260,6 +260,14 @@ can just span multiple lines"`,
 		msg:  `string: '\033'`,
 		want: message{String: "\033"},
 	}, {
+		desc: "StringStripCarriageReturn",
+		msg:  "string:'a\r\nb'",
+		want: message{String: "a\nb"},
+	}, {
+		desc: "StringTab",
+		msg:  "string:'\t'",
+		want: message{String: "\t"},
+	}, {
 		desc: "Message",
 		msg:  `message { field: 10 }`,
 		want: message{Message: &nestedMessage{Field: 10}},
@@ -408,6 +416,12 @@ func TestUnmarshal_Invalid(t *testing.T) {
 		desc: "StringBadUnicode",
 		msg:  `string:"\ugggg"`,
 	}, {
+		desc: "StringControlCharacter",
+		msg:  "string:'\a'",
+	}, {
+		desc: "StringCarriageReturnNotFollowedByNewline",
+		msg:  "string:'\r'",
+	}, {
 		desc: "UnterminatedString",
 		msg:  `string: '`,
 	}, {