Skip to content

ThingTemplate: Add null check for ModuleData after creation - #112

Open
seer-by-sentry[bot] wants to merge 1 commit into
mainfrom
seer/fix/module-data-null-check
Open

ThingTemplate: Add null check for ModuleData after creation#112
seer-by-sentry[bot] wants to merge 1 commit into
mainfrom
seer/fix/module-data-null-check

Conversation

@seer-by-sentry

@seer-by-sentry seer-by-sentry Bot commented Nov 6, 2025

Copy link
Copy Markdown

Fixes CLIENT-RP. The issue was that: INI parsing attempts to load an unregistered module name, causing ModuleFactory to return NULL, which is immediately dereferenced.

  • Added a null check for the ModuleData pointer returned by TheModuleFactory->newModuleDataFromINI.
  • If the ModuleData pointer is null, a debug crash is triggered with detailed information about the error, including the line number, filename, module name, and thing template name.
  • An INI_INVALID_DATA exception is thrown to halt further processing.

This fix was generated by Seer in Sentry, triggered automatically. 👁️ Run ID: 2463556

Not quite right? Click here to continue debugging with Seer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants