Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.Sign up
[dev.icinga.com #2197] Revamp the detection of embedded perl usage directive "# icinga: +epn" #823
This issue has been migrated from Redmine: https://dev.icinga.com/issues/2197
Created by mfriedrich on 2011-12-14 08:14:40 +00:00
2012-07-06 16:33:01 +00:00 by mfriedrich 9707f13
2012-08-19 17:33:30 +00:00 by mfriedrich 6009a11
2012-08-21 15:35:04 +00:00 by mfriedrich eeb8cde
2012-08-21 15:36:25 +00:00 by mfriedrich b513751
Updated by mfriedrich on 2012-07-06 16:28:38 +00:00
patch is revamping the detection of the first 10 lines containing the "#icinga: +epn" line.
previously, the core would have read only the first 80 characters from each line via fgets, which could be to less. the inner handling is changed to 256 chars, which should be sufficient. removing the strtok on ':' is also a good idea, as this can be done via the strstr pointer match on 'epn' as well, calculating the return value based on the character ('+' means embedded perl enabled).