From 54ad45c870e6e7d1bc08da24368d657acb888d27 Mon Sep 17 00:00:00 2001 From: Brian Gough Date: Mon, 22 May 2017 14:58:51 +0100 Subject: [PATCH] add null byte check for ranges --- services/document-updater/app/coffee/RedisManager.coffee | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/services/document-updater/app/coffee/RedisManager.coffee b/services/document-updater/app/coffee/RedisManager.coffee index 8be17c647d..e3bae7d56f 100644 --- a/services/document-updater/app/coffee/RedisManager.coffee +++ b/services/document-updater/app/coffee/RedisManager.coffee @@ -181,6 +181,10 @@ module.exports = RedisManager = if error? logger.error {err: error, doc_id}, error.message return callback(error) + if ranges? and ranges.indexOf("\u0000") != -1 + error = new Error("null bytes found in ranges") + logger.error err: error, doc_id: doc_id, ranges: ranges, error.message + return callback(error) multi = rclient.multi() multi.eval setScript, 1, keys.docLines(doc_id:doc_id), newDocLines # index 0 multi.set keys.docVersion(doc_id:doc_id), newVersion # index 1