The return value is unclear:

• The old API treats zero as false (parameter not handled)
• The new API expects an error code (zero is success).
• To add some fun, any non-zero result is passed to ERR_PTR() and used as a pointer for further parsing. Well, (char*)1 is neither IS_ERR_OR_NULL() nor a valid pointer…