Don't update env_x/y if no points were read.

Clang static analysis reports an 'Assigned value is garbage or
undefined' for this. If no points were read from the file, the
`n_points` variable is 0 and `env_x[n_points-1]` performs an
out-of-bounds access.
This commit is contained in:
Reece H. Dunn
2017-09-13 07:13:29 +01:00
parent 19b2bebc02
commit 4a882e21d3
+4 -2
View File
@@ -1401,8 +1401,10 @@ static int LoadEnvelope2(FILE *f, const char *fname)
n_points++;
}
}
env_x[n_points] = env_x[n_points-1];
env_y[n_points] = env_y[n_points-1];
if (n_points > 0) {
env_x[n_points] = env_x[n_points-1];
env_y[n_points] = env_y[n_points-1];
}
ix = -1;
ix2 = 0;