Clean up.
mount.c: In function âparse_optâ:
mount.c:354: warning: conversion to âsize_tâ from âintâ may change the sign of the result
mount.c:354: warning: conversion to âintâ from âsize_tâ may change the sign of the result
mount.c:359: warning: conversion to âsize_tâ from âintâ may change the sign of the result
mount.c:359: warning: conversion to âintâ from âsize_tâ may change the sign of the result
mount.c: In function âparse_optsâ:
mount.c:374: warning: conversion to âintâ from âsize_tâ may change the sign of the result
mount.c:377: warning: conversion to âsize_tâ from âintâ may change the sign of the result
Character string lengths are usually size_t anyway. We can easily
avoid the implicit type cast here.
Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx>
---
utils/mount/mount.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/utils/mount/mount.c b/utils/mount/mount.c
index 1c6a88f..2909595 100644
--- a/utils/mount/mount.c
+++ b/utils/mount/mount.c
@@ -337,7 +337,7 @@ static void mount_usage(void)
printf(_("\tnfsoptions\tRefer to mount.nfs(8) or nfs(5)\n\n"));
}
-static void parse_opt(const char *opt, int *mask, char *extra_opts, int len)
+static void parse_opt(const char *opt, int *mask, char *extra_opts, size_t len)
{
const struct opt_map *om;
@@ -371,7 +371,7 @@ static void parse_opts(const char *options, int *flags, char **extra_opts)
if (options != NULL) {
char *opts = xstrdup(options);
char *opt, *p;
- int len = strlen(opts) + 1; /* include room for a null */
+ size_t len = strlen(opts) + 1; /* include room for a null */
int open_quote = 0;
*extra_opts = xmalloc(len);
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
