From 95d3cb8d4ff2e3e8fdc186f2ebf617fd29ddfdec Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Thu, 29 Sep 2011 20:00:58 +0200 Subject: Use explicit base 10 for converting strings to integers https://fedorahosted.org/sssd/ticket/1013 --- src/db/sysdb.c | 2 +- src/db/sysdb_ops.c | 2 +- src/providers/ldap/ldap_id.c | 4 ++-- src/providers/proxy/proxy_id.c | 4 ++-- src/responder/common/negcache.c | 4 ++-- src/tools/sss_debuglevel.c | 2 +- 6 files changed, 9 insertions(+), 9 deletions(-) (limited to 'src') diff --git a/src/db/sysdb.c b/src/db/sysdb.c index eafa7fdc..1a326030 100644 --- a/src/db/sysdb.c +++ b/src/db/sysdb.c @@ -378,7 +378,7 @@ int sysdb_attrs_get_uint32_t(struct sysdb_attrs *attrs, const char *name, } errno = 0; - val = strtouint32((const char *) el->values[0].data, &endptr, 0); + val = strtouint32((const char *) el->values[0].data, &endptr, 10); if (errno != 0) return errno; if (*endptr) return EINVAL; diff --git a/src/db/sysdb_ops.c b/src/db/sysdb_ops.c index 522bbb08..f4418e4e 100644 --- a/src/db/sysdb_ops.c +++ b/src/db/sysdb_ops.c @@ -60,7 +60,7 @@ static uint32_t get_attr_as_uint32(struct ldb_message *msg, const char *attr) } errno = 0; - l = strtoll((const char *)v->data, NULL, 0); + l = strtoll((const char *)v->data, NULL, 10); if (errno) { return (uint32_t)-1; } diff --git a/src/providers/ldap/ldap_id.c b/src/providers/ldap/ldap_id.c index 2d8aefff..46f68897 100644 --- a/src/providers/ldap/ldap_id.c +++ b/src/providers/ldap/ldap_id.c @@ -235,7 +235,7 @@ static void users_get_done(struct tevent_req *subreq) break; case BE_FILTER_IDNUM: - uid = (uid_t) strtouint32(state->name, &endptr, 0); + uid = (uid_t) strtouint32(state->name, &endptr, 10); if (errno || *endptr || (state->name == endptr)) { tevent_req_error(req, errno ? errno : EINVAL); return; @@ -478,7 +478,7 @@ static void groups_get_done(struct tevent_req *subreq) break; case BE_FILTER_IDNUM: - gid = (gid_t) strtouint32(state->name, &endptr, 0); + gid = (gid_t) strtouint32(state->name, &endptr, 10); if (errno || *endptr || (state->name == endptr)) { tevent_req_error(req, errno ? errno : EINVAL); return; diff --git a/src/providers/proxy/proxy_id.c b/src/providers/proxy/proxy_id.c index ba45d7a4..24284759 100644 --- a/src/providers/proxy/proxy_id.c +++ b/src/providers/proxy/proxy_id.c @@ -1080,7 +1080,7 @@ void proxy_get_account_info(struct be_req *breq) break; case BE_FILTER_IDNUM: - uid = (uid_t) strtouint32(ar->filter_value, &endptr, 0); + uid = (uid_t) strtouint32(ar->filter_value, &endptr, 10); if (errno || *endptr || (ar->filter_value == endptr)) { return proxy_reply(breq, DP_ERR_FATAL, EINVAL, "Invalid attr type"); @@ -1102,7 +1102,7 @@ void proxy_get_account_info(struct be_req *breq) ret = get_gr_name(breq, ctx, sysdb, domain, ar->filter_value); break; case BE_FILTER_IDNUM: - gid = (gid_t) strtouint32(ar->filter_value, &endptr, 0); + gid = (gid_t) strtouint32(ar->filter_value, &endptr, 10); if (errno || *endptr || (ar->filter_value == endptr)) { return proxy_reply(breq, DP_ERR_FATAL, EINVAL, "Invalid attr type"); diff --git a/src/responder/common/negcache.c b/src/responder/common/negcache.c index d8c4c269..3926574a 100644 --- a/src/responder/common/negcache.c +++ b/src/responder/common/negcache.c @@ -90,7 +90,7 @@ static int sss_ncache_check_str(struct sss_nc_ctx *ctx, char *str, int ttl) } errno = 0; - timestamp = strtoull((const char *)data.dptr, &ep, 0); + timestamp = strtoull((const char *)data.dptr, &ep, 10); if (errno != 0 || *ep != '\0') { /* Malformed entry, remove it and return no entry */ expired = true; @@ -330,7 +330,7 @@ static int delete_permanent(struct tdb_context *tdb, } errno = 0; - timestamp = strtoull((const char *)data.dptr, &ep, 0); + timestamp = strtoull((const char *)data.dptr, &ep, 10); if (errno != 0 || *ep != '\0') { /* Malformed entry, remove it */ remove_key = true; diff --git a/src/tools/sss_debuglevel.c b/src/tools/sss_debuglevel.c index 45723ec2..38262dda 100644 --- a/src/tools/sss_debuglevel.c +++ b/src/tools/sss_debuglevel.c @@ -369,7 +369,7 @@ pid_t parse_pid(const char *strpid) char *endptr; errno = 0; - value = strtol(strpid, &endptr, 0); + value = strtol(strpid, &endptr, 10); if ((errno != 0) || (endptr == strpid) || ((*endptr != '\0') && (*endptr != '\n'))) { return 0; -- cgit