I think the painter might still be active when you call end(), although it should not crash.
end() is called anyway in the destructor, so there is no need to actually call it manually.