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? It's slow and generally not the kind of thing we want to do.