diff options
| author |  Lubomir I. Ivanov <neolit123@gmail.com> | 2017-06-14 01:45:18 +0300 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2017-06-15 14:48:49 -0700 |
| commit | 27deb317b00aa61bedea8d781d8cff8be9a3720e (patch) | |
| tree | 4b5ba530f10b702ec4b779e4a269022ffcd16963 /core | |
| parent | 85021b94b163e955a3b7d038d3652ecda11fe7eb (diff) | |
| download | subsurface-27deb317b00aa61bedea8d781d8cff8be9a3720e.tar.gz | |
tankinfomodel.cpp: clamp row index to [0 - MAX_TANK_INFO]
MAX_TANK_INFO is a new macro in dive.h to define the
maximum number of tank_info_t objects.
TankInfoModel's data() and setData() now check for valid
row indexes before accessing the tank_info[] array directly.
Without this patch TankInfoMode::data() can cause a SIGSEGV.
Reported-by: Pedro Neves <nevesdiver@gmail.com>
Signed-off-by: Lubomir I. Ivanov <neolit123@gmail.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'core')
| -rw-r--r-- | core/dive.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/core/dive.h b/core/dive.h index c65d3ffc5..2565b4af0 100644 --- a/core/dive.h +++ b/core/dive.h @@ -296,6 +296,7 @@ struct divecomputer { #define MAX_CYLINDERS (20) #define MAX_WEIGHTSYSTEMS (6) +#define MAX_TANK_INFO (100) #define W_IDX_PRIMARY 0 #define W_IDX_SECONDARY 1 @@ -923,7 +924,7 @@ struct tank_info_t { const char *name; int cuft, ml, psi, bar; }; -extern struct tank_info_t tank_info[100]; +extern struct tank_info_t tank_info[MAX_TANK_INFO]; struct ws_info_t { const char *name; |