Fix null pointer return in net_udp (#355)

* fix crash seen in node_exporter Signed-off-by: Hubert Chen <[email protected]>
prometheus · Feb 8, 2021 · ded1478 · ded1478
1 parent 8c052e3
commit ded1478
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/net_udp.go b/net_udp.go
@@ -38,17 +38,13 @@ func (fs FS) NetUDP6() (NetUDP, error) {
 // NetUDPSummary returns already computed statistics like the total queue lengths
 // for UDP datagrams read from /proc/net/udp.
 func (fs FS) NetUDPSummary() (*NetUDPSummary, error) {
-	n, err := newNetUDPSummary(fs.proc.Path("net/udp"))
-	n1 := NetUDPSummary(*n)
-	return &n1, err
+	return newNetUDPSummary(fs.proc.Path("net/udp"))
 }
 
 // NetUDP6Summary returns already computed statistics like the total queue lengths
 // for UDP datagrams read from /proc/net/udp6.
 func (fs FS) NetUDP6Summary() (*NetUDPSummary, error) {
-	n, err := newNetUDPSummary(fs.proc.Path("net/udp6"))
-	n1 := NetUDPSummary(*n)
-	return &n1, err
+	return newNetUDPSummary(fs.proc.Path("net/udp6"))
 }
 
 // newNetUDP creates a new NetUDP{,6} from the contents of the given file.