Hello,
imo this PR is not correct, i.e., free(tmporig); doesn't work since tmporig has moved and doesn't point to the start of the buffer.
Since the buffer pointed by orig parameter is not modified inside the function, there is no need to duplicate it.
So, instead of doing

    char *tmporig = malloc(strlen(orig) + 1); // Copy of orig that we work with
    tmporig = strcpy(tmporig, orig);

it should be enough doing instead

    char *tmporig = orig;

so there is no need to perform a free() in the end, and no memory leak is created.

Thanks for catching this, @fralken! You're correct. As I said on the issue comment, I think we can skip copying the original string completely. Updated PR to reflect that.

Hello, are you planning to publish a new release with these fixes?

Released - v.1.4.0.