From: Alejandro Colomar <alx@kernel.org>
Date: Tue, 9 Jan 2024 20:39:43 +0000 (+0100)
Subject: src/faillog.c: Simplify, by calling str2sh() instead of str2sl()
X-Git-Tag: 4.17.0-rc1~151
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=ab9f4da83f0a671d5acd8705c7790e49227db1c8;p=thirdparty%2Fshadow.git

src/faillog.c: Simplify, by calling str2sh() instead of str2sl()

Reviewed-by: "Serge E. Hallyn" <serge@hallyn.com>
Signed-off-by: Alejandro Colomar <alx@kernel.org>
---

diff --git a/src/faillog.c b/src/faillog.c
index 75910bb60..23a9c6c53 100644
--- a/src/faillog.c
+++ b/src/faillog.c
@@ -557,16 +557,12 @@ int main (int argc, char **argv)
 				break;
 			case 'm':
 			{
-				long  lmax;
-
-				if (   (str2sl(&lmax, optarg) == -1)
-				    || ((long)(short) lmax != lmax)) {
+				if (str2sh(&fail_max, optarg) == -1) {
 					fprintf (stderr,
 					         _("%s: invalid numeric argument '%s'\n"),
 					         Prog, optarg);
 					exit (E_BAD_ARG);
 				}
-				fail_max = lmax;
 				mflg = true;
 				break;
 			}