src: fix bad sizeof expression

bnoordhuis · MylesBorins · commit f6ec5fa4e887 · 2017-12-11T15:52:35.000-05:00
It was computing the size of the pointer, not the size of the pointed-to object. Introduced in commit 727b291 ("src,dns: refactor cares_wrap to avoid global state".) PR-URL: #17014 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de>
diff --git a/src/cares_wrap.cc b/src/cares_wrap.cc
@@ -160,7 +160,7 @@ class ChannelWrap : public AsyncWrap {
   }
   inline node_ares_task_list* task_list() { return &task_list_; }
 
-  size_t self_size() const override { return sizeof(this); }
+  size_t self_size() const override { return sizeof(*this); }
 
   static void AresTimeout(uv_timer_t* handle);
 

Original file line number	Diff line number	Diff line change
`@@ -160,7 +160,7 @@ class ChannelWrap : public AsyncWrap {`
`160`	`160`	`}`
`161`	`161`	`inline node_ares_task_list* task_list() { return &task_list_; }`
`162`	`162`
`163`		`- size_t self_size() const override { return sizeof(this); }`
	`163`	`+ size_t self_size() const override { return sizeof(*this); }`
`164`	`164`
`165`	`165`	`static void AresTimeout(uv_timer_t* handle);`
`166`	`166`