@@ -86,7 +86,6 @@ const {
|
86 | 86 | const { |
87 | 87 | isUint32, |
88 | 88 | validateInteger, |
89 | | - validateInt32, |
90 | 89 | validateUint32 |
91 | 90 | } = require('internal/validators'); |
92 | 91 | |
@@ -788,11 +787,7 @@ fs.ftruncate = function(fd, len = 0, callback) {
|
788 | 787 | len = 0; |
789 | 788 | } |
790 | 789 | validateUint32(fd, 'fd'); |
791 | | -// TODO(BridgeAR): This does not seem right. |
792 | | -// There does not seem to be any validation before and if there is any, it |
793 | | -// should work similar to validateUint32 or not have a upper cap at all. |
794 | | -// This applies to all usage of `validateInt32(len, 'len')`. |
795 | | -validateInt32(len, 'len'); |
| 790 | +validateInteger(len, 'len'); |
796 | 791 | len = Math.max(0, len); |
797 | 792 | const req = new FSReqWrap(); |
798 | 793 | req.oncomplete = makeCallback(callback); |
@@ -801,7 +796,7 @@ fs.ftruncate = function(fd, len = 0, callback) {
|
801 | 796 | |
802 | 797 | fs.ftruncateSync = function(fd, len = 0) { |
803 | 798 | validateUint32(fd, 'fd'); |
804 | | -validateInt32(len, 'len'); |
| 799 | +validateInteger(len, 'len'); |
805 | 800 | len = Math.max(0, len); |
806 | 801 | const ctx = {}; |
807 | 802 | binding.ftruncate(fd, len, undefined, ctx); |
|