just for lulz

Ну как сказать.. А как бы написал я при отсутствии Qt? Наверное тоже бы читал ifstream в буфер, тоже бы отсекал вындозный конец строки - в общем принципиально ничем не лучше

Ну в данном случае он бы ничего не нашел - утечек нет, куча не бита. В свое время мой интерес к valgrind быстро охладел когда я познакомился с гораздо более приятными "Instruments" (Memory Leak и др)

Конечно прочитать в std::string было бы аккуратнее. Вообще осуждать др программиста писавшего этот код легко. Но при этом не стоит забывать - "плохой" код реализует не слабый кусок ф-ционала. В аттаче пример файла что он читает. И у меня как-то не возникает желания переписать это чтение с нуля  Ну заменю я массив на контейнер и strlen/strtok на что-то получше - так это пол-процента от всей работы (такова обычная цена умничания  )

Да, если бы найти не удалось - пришлось бы рефакторить, но это тоже совсем не дешево. A "потихонечку" (постепенно улучшать) не всегда есть возможность.