diff options
| author | michael <michael@82007160-df01-0410-b94d-b575c5fd34c7> | 2014-03-14 15:54:32 +0000 |
|---|---|---|
| committer | michael <michael@82007160-df01-0410-b94d-b575c5fd34c7> | 2014-03-14 15:54:32 +0000 |
| commit | de3a7065fa90de18b1224baa0dda9e707872445d (patch) | |
| tree | ce4d11db65370591aca6e280a031f4a1fb1c8b9b /src | |
| parent | 98628b2a4425915acd5fd04c0a8988af4949de07 (diff) | |
- channel_mode.c:chm_limit(): instead of passing an empty buffer to atoi()
we just bail out
git-svn-id: svn://svn.ircd-hybrid.org/svnroot/ircd-hybrid/branches/8.1.x@3154 82007160-df01-0410-b94d-b575c5fd34c7
Diffstat (limited to 'src')
| -rw-r--r-- | src/channel_mode.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/src/channel_mode.c b/src/channel_mode.c index 067e8ad..818c1ee 100644 --- a/src/channel_mode.c +++ b/src/channel_mode.c @@ -1193,7 +1193,6 @@ chm_limit(struct Client *client_p, struct Client *source_p, { unsigned int i; int limit; - char *lstr; if (alev < CHACCESS_HALFOP) { @@ -1210,9 +1209,9 @@ chm_limit(struct Client *client_p, struct Client *source_p, if ((dir == MODE_ADD) && parc > *parn) { - lstr = parv[(*parn)++]; + char *lstr = parv[(*parn)++]; - if ((limit = atoi(lstr)) <= 0) + if (EmptyString(lstr) || (limit = atoi(lstr)) <= 0) return; sprintf(lstr, "%d", limit); @@ -1255,7 +1254,6 @@ chm_key(struct Client *client_p, struct Client *source_p, char **parv, int *errors, int alev, int dir, char c, unsigned int d) { unsigned int i; - char *key; if (alev < CHACCESS_HALFOP) { @@ -1272,14 +1270,14 @@ chm_key(struct Client *client_p, struct Client *source_p, if ((dir == MODE_ADD) && parc > *parn) { - key = parv[(*parn)++]; + char *key = parv[(*parn)++]; if (MyClient(source_p)) fix_key(key); else fix_key_old(key); - if (*key == '\0') + if (EmptyString(key)) return; assert(key[0] != ' '); |