Hi all,<br><br>Attached is a patch so that, when deserializing an icon, PNG icons are properly ignored, and the number of icon entries is updated to reflect this.<br><br>Presently, when mono comes across a PNG icon entry, it doesn&#39;t read it, but it also doesn&#39;t update the size of the icon directory array.. This causes a crash in SaveIconImage(...) when it hits the last entry in the array (which was never read, so ii.iconColors was never initialized etc).<br>
<br>This patch makes sure that if we *do* come accross a PNG icon, we not only skip it, but we make sure it we reduce the icon count properly, so we wont try to write an icon entry that was never read...<br><br>Of course, the real fix would be to read the PNG icon and write it again.. But this should at least help for now!<br>
<br>David.<br><br>