Merge pull request #143 from morisja/ignore_zero_port

Getservbyport should refuse port 0
twosigma · Oct 23, 2024 · 2a32997 · 2a32997
2 parents f6b4edd + bd06f28
commit 2a32997
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/src/handlers.rs b/src/handlers.rs
@@ -204,6 +204,11 @@ impl FromStr for ServiceWithPort {
 
 impl ServiceWithPort {
     fn lookup(&self) -> Result<Option<Service>> {
+        //issue-142
+        //port 0 lookups to sssd return ENOMEM
+        if self.port == 0 {
+            return Ok(None);
+        }
         let proto = match &self.proto {
             Some(p) => Some(CString::new(p.clone())?),
             None => None,