issue-72 - Correct datatype, add test, originally reported by max-pri…

…vatevoid
twosigma · Oct 21, 2024 · 734cbda · 734cbda
1 parent f6b4edd
commit 734cbda
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/ci/test_nsncd.sh b/ci/test_nsncd.sh
@@ -8,7 +8,7 @@ rc=0
 # basic lookups
 getent passwd nsncdtest || rc=1
 getent group nsncdtest || rc=1
-
+id bug72 | grep 2709991565 || rc=1
 # we expect all of these to succeed 
 for i in $(seq 1 100); do
 	getent services 65000 || rc=1

diff --git a/ci/test_nspawn.sh b/ci/test_nspawn.sh
@@ -19,6 +19,8 @@ sudo sed '$ a netgroup: files' -i /etc/nsswitch.conf
 rc=0
 
 sudo useradd nsncdtest 
+sudo groupadd bug72 --gid 2709991565
+sudo useradd bug72 --gid 2709991565
 cp /etc/services ./services
 
 # simple service lookups

diff --git a/src/handlers.rs b/src/handlers.rs
@@ -949,7 +949,7 @@ fn serialize_initgroups(groups: Vec<Gid>) -> Result<Vec<u8>> {
 
     result.extend_from_slice(header.as_slice());
     for group in groups.iter() {
-        result.extend_from_slice(&i32::to_ne_bytes(group.as_raw().try_into()?));
+        result.extend_from_slice(&u32::to_ne_bytes(group.as_raw().try_into()?));
     }
 
     Ok(result)