X-Git-Url: https://ruderich.org/simon/gitweb/?a=blobdiff_plain;f=group.go;h=126ebb0dc349e0127c4ff14f3b61f48cb99cc737;hb=7adc7f83eddd37aa372aa10159128b6d16c4320a;hp=0ad86711431d068ff0922ceace8c39cdba01a1a0;hpb=13b90749eb1b3547bbfbb47fe142cb96a17234b5;p=nsscash%2Fnsscash.git

diff --git a/group.go b/group.go
index 0ad8671..126ebb0 100644
--- a/group.go
+++ b/group.go
@@ -62,9 +62,15 @@ func toKey(g Group) GroupKey {
 func ParseGroups(r io.Reader) ([]Group, error) {
 	var res []Group
 
-	s := bufio.NewScanner(r)
-	for s.Scan() {
-		t := s.Text()
+	s := bufio.NewReader(r)
+	for {
+		t, err := s.ReadString('\n')
+		if err != nil {
+			if err == io.EOF {
+				break
+			}
+			return nil, err
+		}
 
 		x := strings.Split(t, ":")
 		if len(x) != 4 {
@@ -76,6 +82,9 @@ func ParseGroups(r io.Reader) ([]Group, error) {
 			return nil, errors.Wrapf(err, "invalid gid in line %q", t)
 		}
 
+		// ReadString() contains the delimiter
+		x[3] = strings.TrimSuffix(x[3], "\n")
+
 		var members []string
 		// No members must result in empty slice, not slice with the
 		// empty string
@@ -89,15 +98,11 @@ func ParseGroups(r io.Reader) ([]Group, error) {
 			Members: members,
 		})
 	}
-	err := s.Err()
-	if err != nil {
-		return nil, err
-	}
 
 	return res, nil
 }
 
-func SerializeGroup(g Group) []byte {
+func SerializeGroup(g Group) ([]byte, error) {
 	le := binary.LittleEndian
 
 	// Concatenate all (NUL-terminated) strings and store the offsets
@@ -153,7 +158,7 @@ func SerializeGroup(g Group) []byte {
 	// struct are 8 byte aligned
 	alignBufferTo(&res, 8)
 
-	return res.Bytes()
+	return res.Bytes(), nil
 }
 
 func SerializeGroups(w io.Writer, grs []Group) error {
@@ -163,7 +168,11 @@ func SerializeGroups(w io.Writer, grs []Group) error {
 	for _, g := range grs {
 		// TODO: warn about duplicate entries
 		offsets[toKey(g)] = uint64(data.Len())
-		data.Write(SerializeGroup(g))
+		x, err := SerializeGroup(g)
+		if err != nil {
+			return err
+		}
+		data.Write(x)
 	}
 
 	// Copy to prevent sorting from modifying the argument