Change variable type from signed to unsigned int

Signed-off-by: Arnaud Quette <[email protected]>
networkupstools · Mar 15, 2019 · f88b087 · f88b087
1 parent dd6cdbe
commit f88b087
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 11 deletions.
diff --git a/server/netssl.c b/server/netssl.c
@@ -46,7 +46,7 @@ char	*certname = NULL;
 char	*certpasswd = NULL;
 
 #ifdef WITH_CLIENT_CERTIFICATE_VALIDATION
-int certrequest = 0;
+unsigned int certrequest = 0;
 #endif /* WITH_CLIENT_CERTIFICATE_VALIDATION */
 
 static int	ssl_initialized = 0;

diff --git a/server/netssl.h b/server/netssl.h
@@ -32,7 +32,7 @@ extern char	*certfile;
 extern char	*certname;
 extern char	*certpasswd;
 #ifdef WITH_CLIENT_CERTIFICATE_VALIDATION
-extern int certrequest;
+extern unsigned int certrequest;
 #endif /* WITH_CLIENT_CERTIFICATE_VALIDATION */
 
 /* List possible values for certrequested */

diff --git a/server/sstate.c b/server/sstate.c
@@ -341,7 +341,7 @@ const cmdlist_t *sstate_getcmdlist(const upstype_t *ups)
 	return ups->cmdlist;
 }
 
-int sstate_dead(upstype_t *ups, int maxage)
+int sstate_dead(upstype_t *ups, unsigned int maxage)
 {
 	time_t	now;
 	double	elapsed;

diff --git a/server/sstate.h b/server/sstate.h
@@ -47,7 +47,7 @@ const cmdlist_t *sstate_getcmdlist(const upstype_t *ups);
 void sstate_makeinfolist(const upstype_t *ups, char *buf, size_t bufsize);
 void sstate_makerwlist(const upstype_t *ups, char *buf, size_t bufsize);
 void sstate_makeinstcmdlist_t(const upstype_t *ups, char *buf, size_t bufsize);
-int sstate_dead(upstype_t *ups, int maxage);
+int sstate_dead(upstype_t *ups, unsigned int maxage);
 void sstate_infofree(upstype_t *ups);
 void sstate_cmdfree(upstype_t *ups);
 int sstate_sendline(upstype_t *ups, const char *buf);

diff --git a/server/upsd.c b/server/upsd.c
@@ -52,13 +52,13 @@ int	deny_severity = LOG_WARNING;
 	upstype_t	*firstups = NULL;
 
 	/* default 15 seconds before data is marked stale */
-	int	maxage = 15;
+	unsigned int	maxage = 15;
 
 	/* default to 1h before cleaning up status tracking entries */
-	int	tracking_delay = 3600;
+	unsigned int	tracking_delay = 3600;
 
 	/* preloaded to {OPEN_MAX} in main, can be overridden via upsd.conf */
-	int	maxconn = 0;
+	unsigned int	maxconn = 0;
 
 	/* preloaded to STATEPATH in main, can be overridden via upsd.conf */
 	char	*statepath = NULL;
@@ -691,7 +691,7 @@ static void upsd_cleanup(void)
 
 void poll_reload(void)
 {
-	int	ret;
+	unsigned int	ret;
 
 	ret = sysconf(_SC_OPEN_MAX);
 
@@ -922,7 +922,8 @@ int nut_uuid_v4(char *uuid_str)
 /* service requests and check on new data */
 static void mainloop(void)
 {
-	int	i, ret, nfds = 0;
+	unsigned int	i, nfds = 0;
+	int				ret;
 
 	upstype_t	*ups;
 	nut_ctype_t		*client, *cnext;
@@ -1246,7 +1247,7 @@ int main(int argc, char **argv)
 		chroot_start(chroot_path);
 	}
 
-	/* default to system limit (may be overridden in upsd.conf */
+	/* default to system limit (may be overridden in upsd.conf) */
 	maxconn = sysconf(_SC_OPEN_MAX);
 
 	/* handle upsd.conf */

diff --git a/server/upsd.h b/server/upsd.h
@@ -91,7 +91,7 @@ int tracking_is_enabled(void);
 
 /* declarations from upsd.c */
 
-extern int		maxage, maxconn, tracking_delay;
+extern unsigned int	maxage, maxconn, tracking_delay;
 extern char		*statepath, *datapath;
 extern upstype_t	*firstups;
 extern nut_ctype_t	*firstclient;