Fix: add checks to pointer arguments in `m_bmqtool_inpututil`

[678098]

Is there an existing proposal for this?

• I have searched the existing proposals

Is your feature request related to a problem?

Files: m_bmqtool_inpututil.h and m_bmqtool_inpututil.cpp

There are functions with arguments passed as a pointer, with assumption that these pointer values are not null.

Example:

blazingmq/src/applications/bmqtool/m_bmqtool_inpututil.cpp

Lines 50 to 53 in 9b692fe

	void InputUtil::preprocessInput(bsl::string* verb,
	bsl::string* output,
	const bsl::string& input,
	bsl::unordered_set<bsl::string>* keys)

Describe the solution you'd like

Add checks to pointer arguments to functions in m_bmqtool_inpututil.

Example:

blazingmq/src/groups/mqb/mqbmock/mqbmock_cluster.cpp

Lines 380 to 384 in 9b692fe

	Cluster::sendRequest(const Cluster::RequestManagerType::RequestSp& request,
	mqbnet::ClusterNode* target,
	bsls::TimeInterval timeout)
	{
	BSLS_ASSERT_SAFE(target);

Alternatives you considered

No response

[Dyex719]

Seems like a simple fix. Can I take this up?

[678098]

Hi @Dyex719 ! Thanks for your interest, you can proceed with this.
Just a gentle reminder, we have a DCO check for all commits in PR, the simple instruction is here: https://github.com/bloomberg/blazingmq/blob/main/CONTRIBUTING.md#contribution-licensing

[678098]

Hi @Dyex719! Do you plan to solve this issue?

[Dyex719]

How can I test these changes?

[678098]

How can I test these changes?

The PR is good, merged.

Thank you @Dyex719 for your contribution!