Merge Fix nil pointer dereference in events.go handling node removal (#1652)

martin-sucha · martin-sucha · commit b00f7050a5f2 · 2022-09-02T16:25:51.000+02:00
diff --git a/AUTHORS b/AUTHORS
@@ -130,3 +130,4 @@ Ondrej Polakovič <ondrej.polakovic@kiwi.com>
 Sergei Karetnikov <sergei.karetnikov@gmail.com>
 Stefan Miklosovic <smiklosovic@apache.org>
 Adam Burk <amburk@gmail.com>
+Valerii Ponomarov <kiparis.kh@gmail.com>
diff --git a/events.go b/events.go
@@ -240,8 +240,8 @@ func (s *Session) handleRemovedNode(ip net.IP, port int) {
 
 	// we remove all nodes but only add ones which pass the filter
 	host, ok := s.ring.getHostByIP(ip.String())
-	hostID := host.HostID()
 	if ok {
+		hostID := host.HostID()
 		s.ring.removeHost(hostID)
 
 		host.setState(NodeDown)

-Original file line number
+Diff line change
 Sergei Karetnikov <[email protected]>
 Stefan Miklosovic <[email protected]>
 Adam Burk <[email protected]>
 +Valerii Ponomarov <[email protected]>