summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVadim Savchuk <vsavchuk@productengine.com>2010-08-13 18:13:37 +0300
committerVadim Savchuk <vsavchuk@productengine.com>2010-08-13 18:13:37 +0300
commit6b53458b8477e542ddc5cc1f9fb41cc4a98819b1 (patch)
tree1414bd2d7842795f311529c81b1b3cece6f130c5
parent0355910ba5b0ed56aabcf290f940987ad5aec1b4 (diff)
EXT-8662 FIXED Crash on renaming incomplete inventory items.
I couldn't reproduce this problem, but from what I've seen in the crash reporter, this is a very old bug affecting viewers 1.2x and 2.x. When user tries to rename an incomplete inventory item, viewer crashes. The fix is to disallow renaming incomplete items. Reviewed by Seraph at https://codereview.productengine.com/secondlife/r/857/ --HG-- branch : product-engine
-rw-r--r--indra/newview/llinventorybridge.cpp6
1 files changed, 6 insertions, 0 deletions
diff --git a/indra/newview/llinventorybridge.cpp b/indra/newview/llinventorybridge.cpp
index 9bbc6514b4..4e1274645f 100644
--- a/indra/newview/llinventorybridge.cpp
+++ b/indra/newview/llinventorybridge.cpp
@@ -1270,6 +1270,12 @@ BOOL LLItemBridge::isItemRenameable() const
{
return FALSE;
}
+
+ if (!item->isFinished()) // EXT-8662
+ {
+ return FALSE;
+ }
+
return (item->getPermissions().allowModifyBy(gAgent.getID()));
}
return FALSE;