A memory leak point on Orch when addExecutor

[Pterosaur]

An executor was added into the m_comsumerMap, but this executor's pointer will not be released in the destructor of Orch. The lifetime of executor is ambiguous.

sonic-swss/orchagent/portsorch.cpp

Line 707 in e8d81b4

Orch::addExecutor(new Consumer(new SubscriberStateTable(stateDb, STATE_TRANSCEIVER_INFO_TABLE_NAME, TableConsumable::DEFAULT_POP_BATCH_SIZE, 0), this, STATE_TRANSCEIVER_INFO_TABLE_NAME));

sonic-swss/orchagent/orch.cpp

Lines 794 to 799 in e8d81b4

	void Orch::addExecutor(Executor* executor)
	{
	auto inserted = m_consumerMap.emplace(std::piecewise_construct,
	std::forward_as_tuple(executor->getName()),
	std::forward_as_tuple(executor));

sonic-swss/orchagent/orch.h

Line 226 in e8d81b4

virtual ~Orch() = default;

[Pterosaur]

Maybe to use a shared pointer to manage the executor is better.