Каждое выделение памяти должно быть сопряжено ровно с одним освобождением. Когда эта функция заканчивается, память, на которую указывает newBinaryNum, не была освобождена, и адрес этой памяти нигде не сохраняется. Следовательно, утечка памяти.
Поскольку вы не используете никаких преимуществ динамического выделения для этого BinaryNum объекта, не используйте динамическое выделение. Просто объявите вашу переменную как BinaryNum newBinaryNum;. Компилятор не только будет обрабатывать потребности этого объекта в памяти, но и ваш код сможет извлечь выгоду из оптимизации возвращаемого значения .
(Кроме того, вам следует опираться на стандартную библиотекунемного больше. Ваши манипуляции с binaryAry имитируют функциональность std::vector. Зачем заново изобретать колесо? В этой функции вы должны сосредоточиться на классе BinaryNum, поэтому делегируйте детали управления памятьюв стандартную библиотеку.)