Skip to content

Commit 8c3637c

Browse files
authored
stream: fix error-path function call
The `onFinish()` function takes a single argument. The two extra arguments passed here are already in the function scope, and may result in the error being mishandled. PR-URL: #41433 Reviewed-By: Robert Nagy <ronagy@icloud.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
1 parent c4194c0 commit 8c3637c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/internal/streams/writable.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -712,7 +712,7 @@ function callFinal(stream, state) {
712712
}
713713
}
714714
} catch (err) {
715-
onFinish(stream, state, err);
715+
onFinish(err);
716716
}
717717

718718
state.sync = false;

0 commit comments

Comments
 (0)