[rt #69243] use names for i_tags_addn() instead of ST(n) and handle magic

The code checked SvOK() without SvGETMAGIC()

diff --git a/Imager.xs b/Imager.xs
index 53e91852d34fffada6ee37f4fb670e9f6bd21839..3dc6d0b25c39e2a20824216027c1316f8ac7e749 100644 (file)
--- a/Imager.xs
+++ b/Imager.xs
@@ -3630,16 +3630,18 @@ i_img_to_drgb(im)
        Imager::ImgRaw im
 
 undef_int
        Imager::ImgRaw im
 
 undef_int

-i_tags_addn(im, name, code, idata)
+i_tags_addn(im, name_sv, code, idata)

         Imager::ImgRaw im
         Imager::ImgRaw im

+       SV *name_sv

         int     code
         int     idata
       PREINIT:
         char *name;
         STRLEN len;
       CODE:
         int     code
         int     idata
       PREINIT:
         char *name;
         STRLEN len;
       CODE:

-        if (SvOK(ST(1)))
-          name = SvPV(ST(1), len);
+        SvGETMAGIC(name_sv);
+        if (SvOK(name_sv))
+          name = SvPV_nomg(name_sv, len);

         else
           name = NULL;
         RETVAL = i_tags_addn(&im->tags, name, code, idata);
         else
           name = NULL;
         RETVAL = i_tags_addn(&im->tags, name, code, idata);