@xadhoom commented on this pull request.
LM_ERR("body string too short!\n");
+ return NULL; + } + version_start = strstr(body->s + 30, "version="); + if(!version_start) { + LM_ERR("version string not found!\n"); + return NULL; + } + version_start += 9; + + /* safety check for placeholder - if it is body not set by the module, + * don't update the version */ + if(strncmp(version_start, "00000000000"", 12) != 0) + return NULL; + + version_len = snprintf(version, MAX_INT_LEN + 2, "%d"", subs->version);
Well, does not change the flow but yes, is useful for debugging, added.