From 0348fcac51046bc369da3003e70f2790586315d1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bruno=20Reni=C3=A9?= Date: Sun, 19 Jan 2014 16:40:39 +0100 Subject: [PATCH] Fix crash on save when icc_profile is None --- PIL/PngImagePlugin.py | 2 +- Tests/images/icc_profile_none.png | Bin 0 -> 4128 bytes Tests/test_file_png.py | 7 +++++++ 3 files changed, 8 insertions(+), 1 deletion(-) create mode 100644 Tests/images/icc_profile_none.png diff --git a/PIL/PngImagePlugin.py b/PIL/PngImagePlugin.py index 7028083d7..f3738bd24 100644 --- a/PIL/PngImagePlugin.py +++ b/PIL/PngImagePlugin.py @@ -618,7 +618,7 @@ def _save(im, fp, filename, chunk=putchunk, check=0): name = p.tags.desc.get("ASCII", p.tags.desc.get("Unicode", p.tags.desc.get("Macintosh", p.tags.desc.get("en", {}).get("US", "ICC Profile")))).encode("latin1", "replace")[:79] except ImportError: name = b"ICC Profile" - data = name + b"\0\0" + zlib.compress(im.info["icc_profile"]) + data = name or b'ICC Profile' + b"\0\0" + zlib.compress(im.info["icc_profile"]) chunk(fp, b"iCCP", data) ImageFile._save(im, _idat(fp, chunk), [("zip", (0,0)+im.size, 0, rawmode)]) diff --git a/Tests/images/icc_profile_none.png b/Tests/images/icc_profile_none.png new file mode 100644 index 0000000000000000000000000000000000000000..9ecff3a97b8eb78026c4ccf27b72039b52311f1e GIT binary patch literal 4128 zcmV+*5Z~{KP)4Tx0F9H&YZO@&hrjCFppYO8w69(amsv=VB$N*{B0kbdY@)Gk%0!bONT>RC zcbS{&np@Rz6cNeBrOq&W@v(5>AK=1;E8Qp&d?AQ?6-5->sI-gfbXV|sR$u+jxxaJH zEiQE0#URir33GmAoLp8;PEE@d&q%SJE;4i#-7pv|6vhCGL7@32K0d($c-Z&-{7<8+ z6uNU@t%TJ3k_uhu_zcu4lT*_mG>|?S1rdmIK-$Fm|3G>s=IcP(j5|XhZUL*@z(gSK z0RwKeSb|Og^i~ozGYOv4b+&d|COK@|W zTQ~DfYsr*Jv(WjX(fF_dR{RHF{%tfqU1>DFT!oJ3a9e)`&gs+100009a7bBm000XU000XU z0RWnu7ytka{z*hZRCt`-S_fEER~G)?R2Zs=f}k-rFeWNuqU+l7u^M%w?rK2A#1>0* zjZv1kVneZ_sMsN535rx}qDzc1+1Pv3pjc2T(tC4Y>Yn?CYQO*wcJp!h28R1?IsZNV z+{Y!xT-2kPH!dmWQu7D-;{N644>I||mF5pJCH7jAgJc_UuY51xl&f(@<2Ek!(Tyer z$#EQHG8xorHUC{Akw8_dfWH5o_-s<^lqSAV_)=YH=&RZ@!B z`>`1C{y@b3`T#0bDSG#PkGQI@WNlN>kskL;lPxbdGD7ISV0f?a#q60=P@vTy;$Q@} z?%IT)oqLe{A_bABB4BUd23n1_smK7A@$7D+fz((0GBhJ&V+8v7BVcO)MvonZe6a-4 zr=qdK+Y3hyg(D?34F@BF(Z#VdiL<7Xs76*)E;BwxI7q!7BC!Zk(mk$D; z@b}#gSND%lKp;;XI|<+QD~ZF0kn|!MM^8n;!Lds3XF`wFp`QPC$Ji( zbJW4CYDuXYGU^u*=%Nxew1lRd{~U2qD>uf>d9yM1i#fz$Ej<$kl*>teB47nQj7jhI z?cRgUK5JoRrufat8}zh#AG5mU<9PArs!!0<|D_6ZM4bpnyEoby4oE=b@=~7)IwoVX zpi8}17`T1DCY8|4(qafw9 zkQ1RQGJ?)7E@;!S1MkeW#7Sj-0j@Q&IQ-w?$>l3s@$35Sz&FX14S`3(cf4xZ_b+}sk&y_O(s`;H3h`l@rNy20z7 zRg7UeDK|xUIY^;8sVvA8vJtYH&e_KxUQE)iCm!1|$bHCqbW~ zLt);^8Y70dz-8D_ayy#epIm(<@R3<@PO_dOcfcYNc7$AcTF(B~A(v|kko@+vcWzM$ z^0j&l`EWQ6Q((aSZmnQ=Jcix*Fp+#O3o!H)L~})%@%rk6IUX<%ma7EM_g5WP0g|#1 zGCUED$w~StrTs?y?eDO*wZYI)AEHMuXFe`2m$>p2j*pZoK$ZVrJ@DrWtAVW2sue~n zxywTpabD z4~cMTX(^U^c;U#kgHzj)SDcBrjA^N z!N>@utZdc%6d~2Rs;#aiECZLT=8rY?HRgR6mzVOW9w0eZNe0IQ2ND06cZ>NNsV65@z%VQGLzvSy5iHx6ES_pWGIy-wS&w`sQp$; zB$J}#!DA#&oDEAlt=8TPpnj2x_zAOM=eGj-VkL5qokp9L3!zQVhOF}&;OJ0tgOB0m z{rj+8_&GG0IWXzk5qIzY3ePW>;nayRJSVB}U9%CU3I&$=EXDnp2UxRqBaR&pg;h%{ z66rh0&C5ssetjSz9hMTePM$c0Gjt$+>sRst$dwD1u<&2YvHw6I9N%)_gZJ9<2`(%C zUd@jvFxQi;Cyt*3X?q*k@9+gjo27aCL3#5Y+Mhazv}Nm{j!%KKl?`5Ay@{6poPieZ zV~{laOG6DvCx-HN4BD(;2u-iIvBAd|-abokJn}ev*KL8X-)gu`bj2qVXQFjWOKjP` z5rMn+A~fO{+S=P;^zb1V>+T9Man57@B4lOfAUHS>NiS01HD>{`bMoLmV-nol-S{!J z-GFhK&uhMf6&cBp7x`~wuJ%X!llvia?118nWFW@RMr$8WSWFy?gppIAD^;T1fo;g% z7Xp!qDXjl774a_a5ZhV7e$OVP6BwiJZ===hDNq#@@;W}rZ7QrREfIL=DI0*zQ;d6EZ-F zZZl!MXfElX7%9^i!2WO`WbN#b=sF9w{woM*SIEd{Ub=W0b7sxQ!H~V^)9+n`9zKGg zuMXgwZ^Llw_C2gzvI-twevV5we!;RO|KwA?)jk_Aa>Nj<+O!iMb7#P*$2%DBJ{>!D zY($Tq@1R7fF?2I^D5-+i$u}c z3uw9cGn8Dv4Nc)o*ev=SnZCPF`XCNgp3_lu<|i^f7O?!-6{_b6upBlBGiT0+m8mKG z{MYg!TtL8noI3L(f_7~{bo3AK-?AHyT{>X{iL$~}fvvtax%gWT{`pEbT@9^K?`#;m=MSoEi4f8@F^s$NKLNHl4$AF zxg+8dl2Ab6+s3{PMvNQ=GqU75T`fA9_V{|Od^Q9CB0fphmnEzs(i2s38j`1Zz}9a$ zjC*xAJR>(z`sf+b<}QZa*IOZbvmNhB^rSDmdtnRGNi$c#^io$=%3$t*ML}v>g9Qdm zJXzez+=rB`JF~9kW7#uiMcFv!Ewx(195z?~ngD`JeR#EAu`6>C`60)VA9ez5BLav^ z65i>v{_M?rp}c+z_8~hcvK