summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniil Baturin <daniil@vyos.io>2021-04-18 12:19:08 +0300
committerDaniil Baturin <daniil@vyos.io>2021-04-18 12:19:08 +0300
commited9a41190997d4a46c1e8efd21ac06aeb0965965 (patch)
tree1c510be916d4e4a35800dd948646eebb68a032a3
parentbde5c58cc02a873df4b98cfc2524edc72d0b14c5 (diff)
downloadvyos-utils-ed9a41190997d4a46c1e8efd21ac06aeb0965965.tar.gz
vyos-utils-ed9a41190997d4a46c1e8efd21ac06aeb0965965.zip
Consistent formatting in validate-value.ml
-rw-r--r--src/validate_value.ml47
1 files changed, 23 insertions, 24 deletions
diff --git a/src/validate_value.ml b/src/validate_value.ml
index 02cd15a..0b4148f 100644
--- a/src/validate_value.ml
+++ b/src/validate_value.ml
@@ -1,36 +1,35 @@
type check = Regex of string | Exec of string
let validate_value buf value_constraint value =
- match value_constraint with
- | Regex s ->
- (try
- let _ = Pcre.exec ~pat:s value in true
- with Not_found -> false)
- | Exec c ->
- (* XXX: Unix.open_process_in is "shelling out", which is a bad idea on multiple levels,
- especially when the input comes directly from the user...
- We should do something about it.
- *)
- let chan = Unix.open_process_in (Printf.sprintf "%s \'%s\' 2>&1" c value) in
- let out = try CCIO.read_all chan with _ -> "" in
- let result = Unix.close_process_in chan in
- match result with
- | Unix.WEXITED 0 -> true
- | Unix.WEXITED 127 ->
- let () = Printf.printf "Could not execute validator %s" c in
- false
- | _ ->
- let () = Buffer.add_string buf out; Buffer.add_string buf "\n" in
- false
+ match value_constraint with
+ | Regex s ->
+ (try let _ = Pcre.exec ~pat:s value in true
+ with Not_found -> false)
+ | Exec c ->
+ (* XXX: Unix.open_process_in is "shelling out", which is a bad idea on multiple levels,
+ especially when the input comes directly from the user...
+ We should do something about it.
+ *)
+ let chan = Unix.open_process_in (Printf.sprintf "%s \'%s\' 2>&1" c value) in
+ let out = try CCIO.read_all chan with _ -> "" in
+ let result = Unix.close_process_in chan in
+ match result with
+ | Unix.WEXITED 0 -> true
+ | Unix.WEXITED 127 ->
+ let () = Printf.printf "Could not execute validator %s" c in
+ false
+ | _ ->
+ let () = Buffer.add_string buf out; Buffer.add_string buf "\n" in
+ false
let value = ref ""
let checks = ref []
let args = [
- ("--regex", Arg.String (fun s -> checks := (Regex s) :: !checks), "Check the value against a regex");
- ("--exec", Arg.String (fun s -> checks := (Exec s) :: !checks), "Check the value against an external command");
- ("--value", Arg.String (fun s -> value := s), "Value to check");
+ ("--regex", Arg.String (fun s -> checks := (Regex s) :: !checks), "Check the value against a regex");
+ ("--exec", Arg.String (fun s -> checks := (Exec s) :: !checks), "Check the value against an external command");
+ ("--value", Arg.String (fun s -> value := s), "Value to check");
]
let usage = Printf.sprintf "Usage: %s [OPTIONS] <number>" Sys.argv.(0)