fix: make sure errors from unmarshalling are caught

Author: achingbrain

src/exporter/file.js

@@ -15,7 +15,14 @@ module.exports = (cid, node, name, path, pathRest, resolve, size, dag, parent, d
     return pull.empty()
   }
 
-  const file = UnixFS.unmarshal(node.data)
+  let file
+
+  try {
+    file = UnixFS.unmarshal(node.data)
+  } catch (error) {
+    return pull.error(error)
+  }
+
   const fileSize = size || file.fileSize()
 
   if (offset < 0) {
@@ -103,7 +110,14 @@ function streamBytes (dag, node, fileSize, offset, length) {
       return pull.empty()
     }
 
-    const file = UnixFS.unmarshal(node.data)
+    let file
+
+    try {
+      file = UnixFS.unmarshal(node.data)
+    } catch (error) {
+      return pull.error(error)
+    }
+
     const nodeHasData = Boolean(file.data && file.data.length)
 
     // handle case where data is present on leaf nodes and internal nodes

src/exporter/resolve.js

@@ -43,6 +43,7 @@ function createResolver (dag, options, depth, parent) {
         if (err) {
           return cb(err)
         }
+
         // const name = item.fromPathRest ? item.name : item.path
         cb(null, resolveItem(cid, node.value, item, options.offset, options.length))
       })
@@ -57,7 +58,14 @@ function createResolver (dag, options, depth, parent) {
   }
 
   function resolve (cid, node, name, path, pathRest, size, dag, parentNode, depth, offset, length) {
-    const type = typeOf(node)
+    let type
+
+    try {
+      type = typeOf(node)
+    } catch (error) {
+      return pull.error(error)
+    }
+
     const nodeResolver = resolvers[type]
     if (!nodeResolver) {
       return pull.error(new Error('Unkown node type ' + type))