From e437d80133c96096460bf9418f17f25a65a2db46 Mon Sep 17 00:00:00 2001
From: wolfbeast <mcwerewolf@gmail.com>
Date: Wed, 25 Jul 2018 12:34:50 +0200
Subject: Use try/catch in PageThumbs writeData to deal with null data from
 caller.

Quick fix for #670
---
 toolkit/components/thumbnails/PageThumbs.jsm | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

(limited to 'toolkit')

diff --git a/toolkit/components/thumbnails/PageThumbs.jsm b/toolkit/components/thumbnails/PageThumbs.jsm
index 9bd3ae4b3..b0affee92 100644
--- a/toolkit/components/thumbnails/PageThumbs.jsm
+++ b/toolkit/components/thumbnails/PageThumbs.jsm
@@ -609,7 +609,15 @@ this.PageThumbsStorage = {
   writeData: function Storage_writeData(aURL, aData, aNoOverwrite) {
     let path = this.getFilePathForURL(aURL);
     this.ensurePath();
-    aData = new Uint8Array(aData);
+    
+    // XXX: We try/catch here since 'null' isn't accepted until we implement
+    // ES2017's new Uint8Array(); allowance.
+    try {
+      aData = new Uint8Array(aData);
+    } catch(e) { 
+      aData = new Uint8Array(0);
+    }
+    
     let msg = [
       path,
       aData,
-- 
cgit v1.2.3