Skip to content

Commit

Permalink
inspector: add missing initialization
Browse files Browse the repository at this point in the history
Add missing initialization reported by coverity. With
proper usage it looks like it will be initialized but
still good to have it initialized to a known state in
case that changes or invalide usage.

The method used to create object ids newObjectId() starts
at 1 so initializing to 0 should be safe and what we get
now when compiled in a way that auto initializes (for example
debug)

Signed-off-by: Michael Dawson <[email protected]>

PR-URL: #43254
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Minwoo Jung <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
  • Loading branch information
mhdawson authored and targos committed Jul 12, 2022
1 parent 8b9f93c commit ffe79ac
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/inspector/tracing_agent.h
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ class TracingAgent : public NodeTracing::Backend {
Environment* env_;
std::shared_ptr<MainThreadHandle> main_thread_;
tracing::AgentWriterHandle trace_writer_;
int frontend_object_id_;
int frontend_object_id_ = 0;
std::shared_ptr<NodeTracing::Frontend> frontend_;
};

Expand Down

0 comments on commit ffe79ac

Please sign in to comment.