Skip to content

Commit

Permalink
Disable new dangerous check introduced in several previous commits
Browse files Browse the repository at this point in the history
It seems that it does not provide anything useful while, say, Klever can
fail with unclear errors due to it (it is another story why these errors
are unclear).
  • Loading branch information
eunovm committed Jul 4, 2022
1 parent 1e5d189 commit 4712285
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions cif.c
Original file line number Diff line number Diff line change
Expand Up @@ -1000,10 +1000,13 @@ static char *concat(const char *first, ...)

/* Indeed, this function is used in different places, not only to concatenate directory and file names, but anyway
its result should be pretty limited. */
/* This check seems to be to dangerous. Anyway, later something should fail if the concatenated string will be
inappropriate.
if (length >= PATH_MAX) {
fprintf(stderr, "Too long concatenated string.\n");
exit(-1);
}
*/

/* Allocate enough memory to store resulted string. */
res = malloc(length + 1);
Expand Down

0 comments on commit 4712285

Please sign in to comment.