Skip to content

Commit 8602da9

Browse files
committed
optimize free calls
1 parent 8b8ddcf commit 8602da9

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/cert.c

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -225,10 +225,10 @@ error_t cert_generate_signed(const char *subject, const uint8_t *serial_number,
225225

226226
/* save the cert as pem */
227227
FsFile *file = fsOpenFile(cert_file_full, FS_FILE_MODE_WRITE);
228+
osFreeMem(cert_file_full);
228229
if (!file)
229230
{
230231
TRACE_ERROR("fsOpenFile failed\r\n");
231-
osFreeMem(cert_file_full);
232232
return ERROR_FAILURE;
233233
}
234234
if (!cert_der_format)
@@ -239,7 +239,6 @@ error_t cert_generate_signed(const char *subject, const uint8_t *serial_number,
239239
{
240240
fsWriteFile(file, cert_der_data, cert_der_size);
241241
}
242-
osFreeMem(cert_file_full);
243242
fsCloseFile(file);
244243
}
245244

@@ -250,13 +249,12 @@ error_t cert_generate_signed(const char *subject, const uint8_t *serial_number,
250249

251250
/* save the private key */
252251
FsFile *file = fsOpenFile(priv_file_full, FS_FILE_MODE_WRITE);
252+
osFreeMem(priv_file_full);
253253
if (!file)
254254
{
255255
TRACE_ERROR("fsOpenFile failed\r\n");
256-
osFreeMem(priv_file_full);
257256
return ERROR_FAILURE;
258257
}
259-
osFreeMem(priv_file_full);
260258
fsWriteFile(file, priv_data, priv_size);
261259
fsCloseFile(file);
262260
}

0 commit comments

Comments
 (0)