Skip to content

Commit 12de1cb

Browse files
authored
Merge pull request #17 from hyperledger/feature/fix-open
Add missing close() calls for files
2 parents 08c58c9 + 778fa52 commit 12de1cb

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

lib/randombytes/random/random.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ int randombytes(unsigned char *p, int len) {
1313
while (completed < len) {
1414
ssize_t result = read(source, p + completed, len - completed);
1515
if (result < 0) {
16+
close(source);
1617
return ED25519_ERROR;
1718
}
1819
completed += result;

lib/randombytes/urandom/urandom.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,12 @@ int randombytes(unsigned char *p, int len) {
99
} else {
1010
ssize_t result = read(source, p, len);
1111
if (result < 0) {
12+
close(source);
1213
return ED25519_ERROR; /* something went wrong */
1314
}
1415
}
1516

17+
close(source);
18+
1619
return ED25519_SUCCESS;
1720
}

0 commit comments

Comments
 (0)