All submissions to this site are governed by Second Life Project Contribution Agreement. By submitting patches and other information using this site, you acknowledge that you have read, understood, and agreed to those terms.
MAINTENANCE ANNOUNCEMENT - JIRA will undergo maintenance starting 1:00am PDT through 3:00am on Saturday 2010.03.20. Please do not enter issues during this time as the system maybe restarted.
So I did some debugging on this. Here's what I hope I've learned. This crash has to do with LLAgent::makeNewOutfit() assuming that default items exist. It pulls items out based on the count (which is generated by checkboxes in dialog.)
So the quick fix would be to not do most of the code in makeNewOutfit if this returns NULL:
LLViewerInventoryItem* item = gInventory.getItem(mWearableEntry[index].mItemID);
Another option might be to add checking in LLMakeOutfitDialog::getIncludedItems() to make sure that requested items to save actually exist.
Or adding code to the floater to only enable checkboxes for items that exist.
Pixel Gausman added a comment - 08/Aug/08 02:42 PM So I did some debugging on this. Here's what I hope I've learned. This crash has to do with LLAgent::makeNewOutfit() assuming that default items exist. It pulls items out based on the count (which is generated by checkboxes in dialog.)
So the quick fix would be to not do most of the code in makeNewOutfit if this returns NULL:
LLViewerInventoryItem* item = gInventory.getItem(mWearableEntry[index].mItemID);
Another option might be to add checking in LLMakeOutfitDialog::getIncludedItems() to make sure that requested items to save actually exist.
Or adding code to the floater to only enable checkboxes for items that exist.
Pixel Gausman added a comment - 08/Aug/08 02:49 PM Stack trace from Visual Studio. Whump, if this is still open after I get back from SIGGRAPH (18th), I can create a fix.
Bug
Resolved
Normal
OGP Beta: Viewer Crashes when attempting to save appearance
So the quick fix would be to not do most of the code in makeNewOutfit if this returns NULL:
LLViewerInventoryItem* item = gInventory.getItem(mWearableEntry[index].mItemID);
Another option might be to add checking in LLMakeOutfitDialog::getIncludedItems() to make sure that requested items to save actually exist.
Or adding code to the floater to only enable checkboxes for items that exist.