mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-02-19 22:16:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

Datatrak import rework: Add another memory size ckeck

Browse source 
As a last minute addition, and for peace of mind and soul, add just
another size check, to run before reading values from buffer.

Signed-off-by: Salvador Cuñat <salvador.cunat@gmail.com>
This commit is contained in:
Salvador Cuñat 2017-05-07 14:26:56 +02:00 • committed by Dirk Hohndel
parent 47dcc964b8
commit 75762e5f16
1 changed files with 5 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

6
core/datatrak.h
View file

@ -44,22 +44,26 @@ static const struct models_table_t g_models[] = {
extern struct sample *add_sample(struct sample *sample, int time, struct divecomputer *dc);
#define JUMP(_ptr, _n) if ((long) (_ptr += _n) > maxbuf) goto bail
#define CHECK(_ptr, _n) if ((long) _ptr + _n > maxbuf) goto bail
#define read_bytes(_n) \
switch (_n) { \
case 1: \
CHECK(membuf, _n); \
tmp_1byte = membuf[0]; \
break; \
case 2: \
CHECK(membuf, _n); \
tmp_2bytes = two_bytes_to_int (membuf[1], membuf[0]); \
break; \
default: \
CHECK(membuf, _n); \
tmp_4bytes = four_bytes_to_long(membuf[3], membuf[2], membuf[1], membuf[0]); \
break; \
} \
JUMP(membuf, _n);
#define read_string(_property) \
CHECK(membuf, tmp_1byte); \
unsigned char *_property##tmp = (unsigned char *)calloc(tmp_1byte + 1, 1); \
_property##tmp = memcpy(_property##tmp, membuf, tmp_1byte);\
_property = (unsigned char *)strcat(to_utf8(_property##tmp), ""); \