Skip to content

Commit 380aaf8

Browse files
committed
warning on uninitialize buffer pointer
1 parent 0bbfeeb commit 380aaf8

File tree

2 files changed

+9
-8
lines changed

2 files changed

+9
-8
lines changed

test/testcases/tst_grow_header.c

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ check_vars(MPI_Comm comm,
5353
MPI_Offset *start,
5454
MPI_Offset *count)
5555
{
56-
int id, j, r, nerrs=0, err, rank, *buf, rec_dim, dimids[3], nvars;
56+
int id, j, r, nerrs=0, err, rank, *buf=NULL, rec_dim, dimids[3], nvars;
5757
MPI_Offset nrecs;
5858

5959
MPI_Comm_rank(comm, &rank);
@@ -111,7 +111,7 @@ check_vars(MPI_Comm comm,
111111
}
112112

113113
err_out:
114-
free(buf);
114+
if (buf != NULL) free(buf);
115115
return err;
116116
}
117117

@@ -186,7 +186,7 @@ check_vars(MPI_Comm comm,
186186
}
187187
#define WRITE_REC_VAR(id) { \
188188
for (i=0; i<count[1] * count[2]; i++) \
189-
buf[i] = rank + i + id * 10 + start[0]; \
189+
buf[i] = rank + i + id * 10 + (int)start[0]; \
190190
err = ncmpi_put_vara_int_all(ncid, varid[id], start, count, buf); \
191191
CHECK_ERR \
192192
}
@@ -196,7 +196,7 @@ tst_fmt(char *filename,
196196
int cmode)
197197
{
198198
int i, rank, nprocs, ncid, err, nerrs=0;
199-
int *buf, dimid[3], varid[16];
199+
int *buf=NULL, dimid[3], varid[16];
200200
MPI_Info info=MPI_INFO_NULL;
201201
MPI_Offset start[3], count[3], increment;
202202

@@ -379,9 +379,10 @@ tst_fmt(char *filename,
379379
exp_extent = 1024 * 5;
380380
CHECK_HEADER_SIZE
381381

382-
err_out:
383382
err = ncmpi_close(ncid); CHECK_ERR
384-
free(buf);
383+
384+
err_out:
385+
if (buf != NULL) free(buf);
385386
MPI_Info_free(&info);
386387

387388
return nerrs;

test/testcases/tst_redefine.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ static int verbose;
5555
static int
5656
check_vars(MPI_Comm comm, int ncid, int *varid)
5757
{
58-
int i, nerrs=0, err, rank, *buf, nvars;
58+
int i, nerrs=0, err, rank, *buf=NULL, nvars;
5959
MPI_Offset start[2], count[2];
6060

6161
MPI_Comm_rank(comm, &rank);
@@ -102,7 +102,7 @@ check_vars(MPI_Comm comm, int ncid, int *varid)
102102
CHECK_VAL(ncid, varid[3], i, buf[i], rank+i+100)
103103

104104
err_out:
105-
free(buf);
105+
if (buf != NULL) free(buf);
106106
return nerrs;
107107
}
108108

0 commit comments

Comments
 (0)