Skip to content

Commit fb24e3b

Browse files
author
Gonzalo Diaz
committed
[REFACTOR] [Hacker Rank] Warmup: Time Conversion solved ✅. Triying to fix sonarlint detected issues.
1 parent eb6d32c commit fb24e3b

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/lib/exercises/src/hackerrank/warmup/time_conversion.c

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,11 @@ char *HACKERRANK_WARMUP_timeConversion(const char *s) {
4848
char *hour_str = HACKERRANK_WARMUP_getFirst(s, 2);
4949

5050
char *time_str = malloc(strlen(s) + 1);
51+
if (time_str == NULL) {
52+
free(meridian);
53+
free(hour_str);
54+
return NULL;
55+
}
5156

5257
strcpy(time_str, s);
5358
char *temp_time_str;
@@ -68,7 +73,6 @@ char *HACKERRANK_WARMUP_timeConversion(const char *s) {
6873
free(meridian);
6974
free(hour_str);
7075
free(time_str);
71-
free(endptr);
7276
return NULL;
7377
} else {
7478
printf("The integer value is: %ld\n", hour);
@@ -85,8 +89,8 @@ char *HACKERRANK_WARMUP_timeConversion(const char *s) {
8589
char *conversion = malloc(BUFFER_MAX_SIZE * sizeof(char));
8690

8791
snprintf(conversion, BUFFER_MAX_SIZE, "%02ld%s", hour, time_str);
88-
free(hour_str);
8992
free(meridian);
93+
free(hour_str);
9094
free(time_str);
9195

9296
return conversion;

0 commit comments

Comments
 (0)