2006-02-22 James Willcox <snorp@snorp.net>

* src/Database.cs (DownloadSong): don't assume that the length
        argument is valid.


svn path=/trunk/daap-sharp/; revision=57188

ChangeLog

@@ -1,3 +1,8 @@
+2006-02-22  James Willcox  <snorp@snorp.net>
+
+	* src/Database.cs (DownloadSong): don't assume that the length
+	argument is valid.
+
 2006-02-21  James Willcox  <snorp@snorp.net>
 
 	* configure.ac: bump to 0.3.1

src/Database.cs

@@ -376,13 +376,12 @@ namespace DAAP {
             try {
                 using (BinaryReader reader = new BinaryReader (response.GetResponseStream ())) {
                     int count = 0;
-
-                    while (count < response.ContentLength) {
-                        byte[] buf = reader.ReadBytes ((int) Math.Min (ChunkLength,
-                                                                       response.ContentLength - ChunkLength));
-                        writer.Write (buf);
-                        count += buf.Length;
-                    }
+                    byte[] buf = new byte[ChunkLength];
+                    
+                    do {
+                        count = reader.Read (buf, 0, ChunkLength);
+                        writer.Write (buf, 0, count);
+                    } while (count == ChunkLength);
                 }
             } finally {
                 writer.Close ();