That should be gv2, not gv1. However, I believe that it's impossible to reach
the warning code, given the structure of the optree that the perl 5
implementation produces, as gv1 and gv2 will never be NULL, and GvIOn() will
always return non-NULL. (Or croak, but that won't return).
if (!gv1 || !io1)
report_evil_fh(gv1, io1, PL_op->op_type);
if (!gv2 || !io2)
- report_evil_fh(gv1, io2, PL_op->op_type);
+ report_evil_fh(gv2, io2, PL_op->op_type);
}
if (io1 && IoIFP(io1))
do_close(gv1, FALSE);