fs: fix stat dev unsigned cast overflow

The `dev_t` is unsigned on Linux, use Integer::NewFromUnsigned to fix cast overflow PR-URL: #16705 Fixes: #16496 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Khaidi Chu <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
nodejs · Nov 26, 2017 · 972ef1a · 972ef1a
1 parent 659c458
commit 972ef1a
Showing 1 changed file with 0 additions and 8 deletions.
diff --git a/src/node_file.cc b/src/node_file.cc
@@ -451,14 +451,6 @@ Local<Value> BuildStatsObject(Environment* env, const uv_stat_t* s) {
 # else
   Local<Value> blksize = Undefined(env->isolate());
 # endif
-#undef X
-
-  // Integers.
-#define X(name)                                                               \
-  Local<Value> name = Integer::New(env->isolate(), s->st_##name);             \
-  if (name.IsEmpty())                                                         \
-    return Local<Object>();                                                   \
-
   X(dev)
   X(mode)
   X(nlink)