diff options
author | Justin Clark-Casey (justincc) | 2012-09-25 21:35:39 +0100 |
---|---|---|
committer | Justin Clark-Casey (justincc) | 2012-09-25 21:35:39 +0100 |
commit | 4fc0cfba3ce1e6545e334f8e34a0e5b45274081e (patch) | |
tree | 9ab2bc50763d2858ccd5632294d6697574bd1d4c /OpenSim/Region/CoreModules/Agent/AssetTransaction/AgentAssetsTransactions.cs | |
parent | Documenting object-related events (diff) | |
download | opensim-SC_OLD-4fc0cfba3ce1e6545e334f8e34a0e5b45274081e.zip opensim-SC_OLD-4fc0cfba3ce1e6545e334f8e34a0e5b45274081e.tar.gz opensim-SC_OLD-4fc0cfba3ce1e6545e334f8e34a0e5b45274081e.tar.bz2 opensim-SC_OLD-4fc0cfba3ce1e6545e334f8e34a0e5b45274081e.tar.xz |
Fix occasional race condition failure when creating new clothing/body parts in the viewer or updating existing assets.
On creating these items, the viewer sends a UDP AssetUploadRequest followed by a CreateInventoryItem.
It was possible for the CreateInventoryItem/UpdateInventoryItem to occasionally outrace the AssetUploadRequest and fail to find an initialized Xfer object, at which point the item create would fail.
So instead we always set up a Xfer object on either the asset or inventory item update request.
This does not introduce a new race because code already exists to delay the item operation until the asset is uploaded if necessary (but this only worked if the xfer object already existed)
Diffstat (limited to 'OpenSim/Region/CoreModules/Agent/AssetTransaction/AgentAssetsTransactions.cs')
-rw-r--r-- | OpenSim/Region/CoreModules/Agent/AssetTransaction/AgentAssetsTransactions.cs | 123 |
1 files changed, 38 insertions, 85 deletions
diff --git a/OpenSim/Region/CoreModules/Agent/AssetTransaction/AgentAssetsTransactions.cs b/OpenSim/Region/CoreModules/Agent/AssetTransaction/AgentAssetsTransactions.cs index b557ffe..bba7b9c 100644 --- a/OpenSim/Region/CoreModules/Agent/AssetTransaction/AgentAssetsTransactions.cs +++ b/OpenSim/Region/CoreModules/Agent/AssetTransaction/AgentAssetsTransactions.cs | |||
@@ -57,39 +57,36 @@ namespace OpenSim.Region.CoreModules.Agent.AssetTransaction | |||
57 | } | 57 | } |
58 | 58 | ||
59 | /// <summary> | 59 | /// <summary> |
60 | /// Return a xfer uploader if one does not already exist. | 60 | /// Return the xfer uploader for the given transaction. |
61 | /// </summary> | 61 | /// </summary> |
62 | /// <remarks> | ||
63 | /// If an uploader does not already exist for this transaction then it is created, otherwise the existing | ||
64 | /// uploader is returned. | ||
65 | /// </remarks> | ||
62 | /// <param name="transactionID"></param> | 66 | /// <param name="transactionID"></param> |
63 | /// <param name="assetID"> | 67 | /// <returns>The asset xfer uploader</returns> |
64 | /// We must transfer the new asset ID into the uploader on creation, otherwise | 68 | public AssetXferUploader RequestXferUploader(UUID transactionID) |
65 | /// we can see race conditions with other threads which can retrieve an item before it is updated with the new | ||
66 | /// asset id. | ||
67 | /// </param> | ||
68 | /// <returns> | ||
69 | /// The xfer uploader requested. Null if one is already in existence. | ||
70 | /// FIXME: This is a bizarre thing to do, and is probably meant to signal an error condition if multiple | ||
71 | /// transfers are made. Needs to be corrected. | ||
72 | /// </returns> | ||
73 | public AssetXferUploader RequestXferUploader(UUID transactionID, UUID assetID) | ||
74 | { | 69 | { |
70 | AssetXferUploader uploader; | ||
71 | |||
75 | lock (XferUploaders) | 72 | lock (XferUploaders) |
76 | { | 73 | { |
77 | if (!XferUploaders.ContainsKey(transactionID)) | 74 | if (!XferUploaders.ContainsKey(transactionID)) |
78 | { | 75 | { |
79 | AssetXferUploader uploader = new AssetXferUploader(this, m_Scene, assetID, m_dumpAssetsToFile); | 76 | uploader = new AssetXferUploader(this, m_Scene, m_dumpAssetsToFile); |
80 | 77 | ||
81 | // m_log.DebugFormat( | 78 | // m_log.DebugFormat( |
82 | // "[AGENT ASSETS TRANSACTIONS]: Adding asset xfer uploader {0} since it didn't previously exist", transactionID); | 79 | // "[AGENT ASSETS TRANSACTIONS]: Adding asset xfer uploader {0} since it didn't previously exist", transactionID); |
83 | 80 | ||
84 | XferUploaders.Add(transactionID, uploader); | 81 | XferUploaders.Add(transactionID, uploader); |
85 | 82 | } | |
86 | return uploader; | 83 | else |
84 | { | ||
85 | uploader = XferUploaders[transactionID]; | ||
87 | } | 86 | } |
88 | } | 87 | } |
89 | 88 | ||
90 | m_log.WarnFormat("[AGENT ASSETS TRANSACTIONS]: Ignoring request for asset xfer uploader {0} since it already exists", transactionID); | 89 | return uploader; |
91 | |||
92 | return null; | ||
93 | } | 90 | } |
94 | 91 | ||
95 | public void HandleXfer(ulong xferID, uint packetID, byte[] data) | 92 | public void HandleXfer(ulong xferID, uint packetID, byte[] data) |
@@ -151,23 +148,11 @@ namespace OpenSim.Region.CoreModules.Agent.AssetTransaction | |||
151 | string description, string name, sbyte invType, | 148 | string description, string name, sbyte invType, |
152 | sbyte type, byte wearableType, uint nextOwnerMask) | 149 | sbyte type, byte wearableType, uint nextOwnerMask) |
153 | { | 150 | { |
154 | AssetXferUploader uploader = null; | 151 | AssetXferUploader uploader = RequestXferUploader(transactionID); |
155 | 152 | ||
156 | lock (XferUploaders) | 153 | uploader.RequestCreateInventoryItem( |
157 | { | 154 | remoteClient, transactionID, folderID, callbackID, |
158 | if (XferUploaders.ContainsKey(transactionID)) | 155 | description, name, invType, type, wearableType, nextOwnerMask); |
159 | uploader = XferUploaders[transactionID]; | ||
160 | } | ||
161 | |||
162 | if (uploader != null) | ||
163 | uploader.RequestCreateInventoryItem( | ||
164 | remoteClient, transactionID, folderID, | ||
165 | callbackID, description, name, invType, type, | ||
166 | wearableType, nextOwnerMask); | ||
167 | else | ||
168 | m_log.ErrorFormat( | ||
169 | "[AGENT ASSET TRANSACTIONS]: Could not find uploader with transaction ID {0} when handling request to create inventory item {1} from {2}", | ||
170 | transactionID, name, remoteClient.Name); | ||
171 | } | 156 | } |
172 | 157 | ||
173 | /// <summary> | 158 | /// <summary> |
@@ -197,69 +182,37 @@ namespace OpenSim.Region.CoreModules.Agent.AssetTransaction | |||
197 | SceneObjectPart part, UUID transactionID, | 182 | SceneObjectPart part, UUID transactionID, |
198 | TaskInventoryItem item) | 183 | TaskInventoryItem item) |
199 | { | 184 | { |
200 | AssetXferUploader uploader = null; | 185 | AssetBase asset = GetTransactionAsset(transactionID); |
201 | 186 | ||
202 | lock (XferUploaders) | 187 | // Only legacy viewers use this, and they prefer CAPS, which |
203 | { | 188 | // we have, so this really never runs. |
204 | if (XferUploaders.ContainsKey(transactionID)) | 189 | // Allow it, but only for "safe" types. |
205 | uploader = XferUploaders[transactionID]; | 190 | if ((InventoryType)item.InvType != InventoryType.Notecard && |
206 | } | 191 | (InventoryType)item.InvType != InventoryType.LSL) |
192 | return; | ||
207 | 193 | ||
208 | if (uploader != null) | 194 | if (asset != null) |
209 | { | 195 | { |
210 | AssetBase asset = GetTransactionAsset(transactionID); | ||
211 | |||
212 | // Only legacy viewers use this, and they prefer CAPS, which | ||
213 | // we have, so this really never runs. | ||
214 | // Allow it, but only for "safe" types. | ||
215 | if ((InventoryType)item.InvType != InventoryType.Notecard && | ||
216 | (InventoryType)item.InvType != InventoryType.LSL) | ||
217 | return; | ||
218 | |||
219 | if (asset != null) | ||
220 | { | ||
221 | // m_log.DebugFormat( | 196 | // m_log.DebugFormat( |
222 | // "[AGENT ASSETS TRANSACTIONS]: Updating item {0} in {1} for transaction {2}", | 197 | // "[AGENT ASSETS TRANSACTIONS]: Updating item {0} in {1} for transaction {2}", |
223 | // item.Name, part.Name, transactionID); | 198 | // item.Name, part.Name, transactionID); |
224 | 199 | ||
225 | asset.FullID = UUID.Random(); | 200 | asset.FullID = UUID.Random(); |
226 | asset.Name = item.Name; | 201 | asset.Name = item.Name; |
227 | asset.Description = item.Description; | 202 | asset.Description = item.Description; |
228 | asset.Type = (sbyte)item.Type; | 203 | asset.Type = (sbyte)item.Type; |
229 | item.AssetID = asset.FullID; | 204 | item.AssetID = asset.FullID; |
230 | 205 | ||
231 | m_Scene.AssetService.Store(asset); | 206 | m_Scene.AssetService.Store(asset); |
232 | } | ||
233 | } | ||
234 | else | ||
235 | { | ||
236 | m_log.ErrorFormat( | ||
237 | "[AGENT ASSET TRANSACTIONS]: Could not find uploader with transaction ID {0} when handling request to update task inventory item {1} in {2}", | ||
238 | transactionID, item.Name, part.Name); | ||
239 | } | 207 | } |
240 | } | 208 | } |
241 | 209 | ||
242 | public void RequestUpdateInventoryItem(IClientAPI remoteClient, | 210 | public void RequestUpdateInventoryItem(IClientAPI remoteClient, |
243 | UUID transactionID, InventoryItemBase item) | 211 | UUID transactionID, InventoryItemBase item) |
244 | { | 212 | { |
245 | AssetXferUploader uploader = null; | 213 | AssetXferUploader uploader = RequestXferUploader(transactionID); |
246 | |||
247 | lock (XferUploaders) | ||
248 | { | ||
249 | if (XferUploaders.ContainsKey(transactionID)) | ||
250 | uploader = XferUploaders[transactionID]; | ||
251 | } | ||
252 | 214 | ||
253 | if (uploader != null) | 215 | uploader.RequestUpdateInventoryItem(remoteClient, transactionID, item); |
254 | { | ||
255 | uploader.RequestUpdateInventoryItem(remoteClient, transactionID, item); | ||
256 | } | ||
257 | else | ||
258 | { | ||
259 | m_log.ErrorFormat( | ||
260 | "[AGENT ASSET TRANSACTIONS]: Could not find uploader with transaction ID {0} when handling request to update inventory item {1} for {2}", | ||
261 | transactionID, item.Name, remoteClient.Name); | ||
262 | } | ||
263 | } | 216 | } |
264 | } | 217 | } |
265 | } | 218 | } \ No newline at end of file |