The outputFile might still be in use from this application, as the stream you opened for it earlier might not quite be closed. If that’s the case, you could go the route that I suggested above for urza, where you save a temporary file and copy it over. Alternatively, you could create a loop that waits for the file to be accessible, but be sure to have a cap for that in case it never becomes accessible.

If it is a permissions problem, you’ll have to require that your application be ran with administrator privileges, or notify the user that the file they are trying to modify is not accessible by them (I’d go with the latter).

On the line output.Save(outputFile);
I’m getting the error ‘Access is Denied’

Any idea what this means?

Are you able to write the new image to a separate file? That is, replace:

// finally, save the new file over the old file
using (Stream outputFile = File.Open(filename, FileMode.Create, FileAccess.Write))
{
output.Save(outputFile);
}

with:
// finally, save the new file over the old file
using (Stream outputFile = File.Open(filename + “.tmp”, FileMode.Create, FileAccess.Write))
{
output.Save(outputFile);
}

If so, a workaround might be to save it to a temporary file (as shown) and override the original file afterwords. This way, you can check the newly created file before overriding the original and causing data loss.

Andrew

I still have the owerflow exception. Even with the cloning. It only works with pictures that allready have some metadata. Thats why I tought it works – I tried it on pictures in windows vista MyPictures/Samples – all files there have ratings or other metadata.. but when I download photo or picture from internet. It has no metadata and the code throws up the exception. It also destroys the file. The file has size 0 B and is not readable.

any ideas? why is it only working with files that allready have metadata?

Would you know what string do I need to pass to SetQuery if I would also like to change the image title and image description in metadata?