Writing behind the allocated memory

[jernst]

Code excerpt:

input = vmalloc(len);
...
input[len] = '\0';

From https://github.com/milesp20/intel_nuc_led/blob/6a3850eadff554053ca7d95e830a624b28c53670/nuc_led.c#L234

That should be:

input = vmalloc(len+1);
...
input[len] = '\0';

[ju2wheels]

This seems oddly similar to my thoughts on #15 wrt to the null character being added to the output on read but I dont write enough C to chase it down.