Fix NPC trading crash

Right, getting pointers to a copied inventory that goes out of scope will cause issues. Why do we even have a copy constructor for inventory, anyway?