From 1534c6f5c3ecc70066a03d0b7df074bcd2e6f48c Mon Sep 17 00:00:00 2001 From: Father Chrysostomos Date: Thu, 28 Nov 2013 16:05:30 -0800 Subject: [PATCH] Revert part of 2efab60d9 As Karl Williamson pointed out in <5296C04E.10103@khwilliamson.com>, sv_setpv will do SvOK_off if Strerror returns null, so this SvOK check is not redundant. --- mg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mg.c b/mg.c index 3f6eb2c..4b1deaf 100644 --- a/mg.c +++ b/mg.c @@ -849,7 +849,7 @@ Perl_magic_get(pTHX_ SV *sv, MAGIC *mg) sv_setnv(sv, (NV)Perl_rc); sv_setpv(sv, os2error(Perl_rc)); } - if (strNE(SvPVX(sv), "")) { + if (SvOK(sv) && strNE(SvPVX(sv), "")) { fixup_errno_string(sv); } # elif defined(WIN32) -- 2.7.4