diff options
author | Denis Kenzior <denkenz@gmail.com> | 2009-12-01 16:05:20 -0600 |
---|---|---|
committer | Denis Kenzior <denkenz@gmail.com> | 2009-12-01 16:05:20 -0600 |
commit | 184eb91aa63f85ccb5df02e1d41cae1a5c8e67db (patch) | |
tree | c0e4eb15b6f0b7b5db467e58c5c75c394d86db15 /drivers/mbmmodem | |
parent | a5149839818851d70bdf888c8cd0a37ce255ba41 (diff) | |
download | ofono-184eb91aa63f85ccb5df02e1d41cae1a5c8e67db.tar.bz2 |
Fix: Make sure we don't overflow the buffer
Diffstat (limited to 'drivers/mbmmodem')
-rw-r--r-- | drivers/mbmmodem/gprs-context.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/mbmmodem/gprs-context.c b/drivers/mbmmodem/gprs-context.c index 778d3bf6..f3f89261 100644 --- a/drivers/mbmmodem/gprs-context.c +++ b/drivers/mbmmodem/gprs-context.c @@ -43,6 +43,9 @@ #define MBM_E2NAP_CONNECTED 1 #define MBM_E2NAP_CONNECTING 2 +#define AUTH_BUF_LENGTH OFONO_GPRS_MAX_USERNAME_LENGTH + \ + OFONO_GPRS_MAX_PASSWORD_LENGTH + 128 + static const char *none_prefix[] = { NULL }; struct gprs_context_data { @@ -132,7 +135,7 @@ static void mbm_gprs_activate_primary(struct ofono_gprs_context *gc, { struct gprs_context_data *gcd = ofono_gprs_context_get_data(gc); struct cb_data *cbd = cb_data_new(cb, data); - char buf[OFONO_GPRS_MAX_APN_LENGTH + 128]; + char buf[AUTH_BUF_LENGTH]; int len; if (!cbd) |