From 1a54d0fa59e0e1c90db1b66907c48a286d5b26a3 Mon Sep 17 00:00:00 2001 From: MaleDong Date: Mon, 22 Apr 2019 09:41:33 +0800 Subject: dgram: change 'this' to 'self' for 'isConnected' The function 'isConnected' is directly called by many 'Socket' instance, so we shouldn't directly use 'this' because 'this' will be the self of function itself, and we should use 'self' as the instance of 'Socket' function. PR-URL: https://github.com/nodejs/node/pull/27338 Reviewed-By: Santiago Gimeno Reviewed-By: Joyee Cheung Reviewed-By: Luigi Pinca Reviewed-By: Colin Ihrig Reviewed-By: Franziska Hinkelmann Reviewed-By: Yongsheng Zhang Reviewed-By: Benjamin Gruenbaum --- lib/dgram.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/dgram.js') diff --git a/lib/dgram.js b/lib/dgram.js index 5a6d5aca2c..b38aa39c21 100644 --- a/lib/dgram.js +++ b/lib/dgram.js @@ -519,7 +519,7 @@ function clearQueue() { function isConnected(self) { try { - this.remoteAddress(); + self.remoteAddress(); return true; } catch { return false; -- cgit v1.2.3