steampipe

mirror of https://github.com/turbot/steampipe.git synced 2025-12-19 18:12:43 -05:00

Files

Nathan Wallace ea6a7a09cc Nil pointer dereference in plugin version check closes #4747 (#4897 )

* Unskip test demonstrating bug #4747: Nil pointer dereference in plugin version check

Removed the GlobalConfig initialization workaround from setupTestEnvironment()
and added TestPluginVersionCheckWithNilGlobalConfig to demonstrate the bug.

The test will panic at runner.go:106 when attempting to access
steampipeconfig.GlobalConfig.PluginVersions without a nil check.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

* Fix #4747: Add nil check for GlobalConfig in version checks

Wraps both CLI and plugin version checks with a nil check for
steampipeconfig.GlobalConfig. This prevents panics when the runner
is invoked before full configuration initialization, such as in
test environments or unusual startup scenarios.

The previous fix only addressed the plugin version check at line 109,
but the CLI version check at line 104 also triggers a panic through
fetchAvailableCLIVersion -> BuildRequestPayload which accesses
GlobalConfig internally.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>