The code

$SNF_fh->close
[download]

closes the filehandle $SNF_fh.

But further up, the call to tempfile returns a filehandle that you store in the variable $fh. You never show where $fh is closed.

If you are not interested in the filehandle and only in the name of the tempfile, immediately closing the filehandle returning from it is the sanest approach:

    my ($fh, $filename) = tempfile( DIR => $self->{Temp_Dir} );
    close $fh; # we only want the filename
[download]