ArgumentException when instantiating bitmap from stream

Question

I've written some code to import content from my Blogger blog. Once I've downloaded all of the HTML content, I go through the image tags and download the corresponding images. In a significant number of cases, System.Drawing.Bitmap.FromStream is throwing an ArgumentException. The URL I'm downloading from looks good and it serves up an image as expected (here's the URL for one of the problem images: http://4.bp.blogspot.com/_tSWCyhtOc38/SgIPcctWRZI/AAAAAAAAAGg/2LLnVPxsogI/s1600-h/IMG_3590.jpg). ``` private static System.Drawing.Image DownloadImage(string source) { System.Drawing.Image image = null; // used to fetch content var client = new HttpClient(); // used to store image data var memoryStream = new MemoryStream(); try { // fetch the image var imageStream = client.GetStreamAsync(source).Result; // instantiate a system.drawing.image from the data image = System.Drawing.Bitmap.FromStream(imageStream, false, false); // save the image data to a memory stream image.Save(memoryStream, im…

Accepted Answer

OK, I found the issue. It turns out that, in some cases, Blogger references an HTML page that renders an image rather than referencing the image itself. So, the response in that case isn't a valid image. I've added code to check the response headers before attempting to save the image data and that's fixed the problem. For the benefit of anyone else who hits this issue, here's the updated code:

```
    private static System.Drawing.Image DownloadImage(string source)
    {
        System.Drawing.Image image = null;

// used to fetch content
        var client = new HttpClient();

// used to store image data
        var memoryStream = new MemoryStream();

try
        {
            // Blogger tacks on a -h to an image Url to link to an HTML page instead
            if (source.Contains("-h/"))
                source = source.Replace("-h/", "/");

// fetch the image
            var response = client.GetAsync(source).Result;
            response.EnsureSuccessStatusCode();

var contentType = response.Content.Headers.ContentType.MediaType;

if (!contentType.StartsWith("image/"))
            {
                Debug.WriteLine(contentType);
                throw new ArgumentException("Specified source did not return an image");
            }

var imageStream = response.Content.ReadAsStreamAsync().Result;

// instantiate a system.drawing.image from the data
            image = System.Drawing.Bitmap.FromStream(imageStream, true, true);

// save the image data to a memory stream
            image.Save(memoryStream, image.RawFormat);
        }
        catch (HttpRequestException exception)
        {
            // sometimes, we'll get a 404 or other unexpected response
            Debug.WriteLine("{0} {1}", exception.Message, source);
        }
        catch (IOException exception)
        {
            Debug.WriteLine("{0} {1}", exception.Message, source);
        }
        catch (ArgumentException exception)
        {
            // sometimes, an image will link to a web page, resulting in this exception
            Debug.WriteLine("{0} {1}", exception.Message, source);
        }
        finally
        {
            // clean up our disposable resources
            client.Dispose();
            memoryStream.Dispose();
        }

return image;
    }
```

ArgumentException when instantiating bitmap from stream

Solution

Problem