simon/real-name
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Ensure gecos string has at least two fields

Browse source 
For a `gecos` string like `,,,`, `clojure.string/split` produces `[]` so `interleave` produces an empty result (it stops on the shortest sequence) and so the `apply assoc` fails with two few arguments.

This ensures that the `gecos` string will always produce a non-empty result from the `split` so the rest works.

Note that `(split "a,,," #",")` produces `["a"]` -- the empty trailing elements are ignored/dropped.
This commit is contained in:
Sean Corfield 2023-06-10 22:58:19 -07:00 committed by GitHub
parent 2bce582c5a
commit 63125d6ee5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/cc/journeyman/real_name/core.clj
View file

@ -33,7 +33,7 @@
"Process this `gecos` field into a map of its sub-fields. See "Process this `gecos` field into a map of its sub-fields. See
https://en.wikipedia.org/wiki/Gecos_field" https://en.wikipedia.org/wiki/Gecos_field"
[gecos] [gecos]
(delimited-record->map gecos #"," [:real-name :address :work-phone :home-phone :other])) (delimited-record->map (str gecos ",?") #"," [:real-name :address :work-phone :home-phone :other]))
(defn process-passwd-line (defn process-passwd-line
"Process this `line` from a passwd file" "Process this `line` from a passwd file"