Changeset 77ed1b6 in subsurface


Ignore:
Timestamp:
Feb 6, 2017, 4:49:24 AM (4 months ago)
Author:
Dirk Hohndel <dirk@…>
Branches:
master
Children:
6fd438e
Parents:
4c74b15
Message:

TestParse?: prevent crashes

Two CSV imports are failing right now; we shouldn't access uninitialized
memory when that happens.

Signed-off-by: Dirk Hohndel <dirk@…>

File:
1 edited

Legend:

Unmodified
Added
Removed
  • tests/testparse.cpp

    r90af2f1 r77ed1b6  
    203203         * time, thus we need to use a static (random) timestamp
    204204         */
    205         struct dive *dive = dive_table.dives[dive_table.nr - 1];
    206         dive->when = 1255152761;
    207         dive->dc.when = 1255152761;
     205        if (dive_table.nr > 0) {
     206                struct dive *dive = dive_table.dives[dive_table.nr - 1];
     207                dive->when = 1255152761;
     208                dive->dc.when = 1255152761;
     209        }
    208210}
    209211
     
    343345                                        params, pnr - 1, "csv"), 0);
    344346
     347                QCOMPARE(dive_table.nr, 1);
    345348                /*
    346349                 * Set artificial but static dive times so the result
     
    348351                 */
    349352
    350                 dive = dive_table.dives[dive_table.nr - 1];
    351                 dive->when = 1255152761 + 7200 * i;
    352                 dive->dc.when = 1255152761 + 7200 * i;
     353                if (dive_table.nr > 0) {
     354                        dive = dive_table.dives[dive_table.nr - 1];
     355                        dive->when = 1255152761 + 7200 * i;
     356                        dive->dc.when = 1255152761 + 7200 * i;
     357                }
    353358        }
    354359
Note: See TracChangeset for help on using the changeset viewer.