From 5e381ec67c1a8d108fced63b76ae44eeb70aa38e Mon Sep 17 00:00:00 2001
From: Latif Khalifa <lkalif@openmetaverse.org>
Date: Sat, 9 Oct 2010 18:01:11 +0200
Subject: [PATCH] Return error code instead of the last byte of the file if
 range is not satisfiable

---
 .../CoreModules/Avatar/Assets/GetTextureModule.cs      | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/OpenSim/Region/CoreModules/Avatar/Assets/GetTextureModule.cs b/OpenSim/Region/CoreModules/Avatar/Assets/GetTextureModule.cs
index d564ee6a10..6545be769d 100644
--- a/OpenSim/Region/CoreModules/Avatar/Assets/GetTextureModule.cs
+++ b/OpenSim/Region/CoreModules/Avatar/Assets/GetTextureModule.cs
@@ -186,7 +186,15 @@ namespace OpenSim.Region.CoreModules.Avatar.ObjectCaps
                 // Range request
                 int start, end;
                 if (TryParseRange(range, out start, out end))
-                {
+                {
+                    // Before clamping end make sure we can satisfy it in order to avoid
+                    // sending back the last byte instead of an error status
+                    if (end >= texture.Data.Length)
+                    {
+                        response.StatusCode = (int)System.Net.HttpStatusCode.RequestedRangeNotSatisfiable;
+                        return;
+                    }
+
                     end = Utils.Clamp(end, 0, texture.Data.Length - 1);
                     start = Utils.Clamp(start, 0, end);
                     int len = end - start + 1;