From 6f838722896279d7531c6a956fbd7cb1c0bb5f3e Mon Sep 17 00:00:00 2001 From: Chris Buechler Date: Fri, 11 Mar 2016 23:46:43 -0600 Subject: Don't try to put the file contents in place if file is empty. Ticket #5982 --- src/etc/inc/pfsense-utils.inc | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) (limited to 'src/etc') diff --git a/src/etc/inc/pfsense-utils.inc b/src/etc/inc/pfsense-utils.inc index e364afb..77ca8e2 100644 --- a/src/etc/inc/pfsense-utils.inc +++ b/src/etc/inc/pfsense-utils.inc @@ -1929,6 +1929,10 @@ function parse_aliases_file($filename, $type = "url", $max_items = -1, $kflc = f * RETURNS an array of ip subnets and ip's or ports and port-ranges, returns NULL upon a error conditions (file not found) */ + if (filesize($filename) == 0) { + log_error(sprintf(gettext("Could not process empty file from alias: %s"), $filename)); + return null; + } $fd = @fopen($filename, 'r'); if (!$fd) { log_error(sprintf(gettext("Could not process aliases from alias: %s"), $filename)); @@ -2179,7 +2183,9 @@ function process_alias_urltable($name, $url, $freq, $forceupdate=false, $validat file_put_contents($urltable_filename, implode("\n", $ports)); } else { $urltable = parse_aliases_file($tmp_urltable_filename, "url", "-1", true); - file_put_contents($urltable_filename, implode("\n", $urltable)); + if (is_array($urltable)) { + file_put_contents($urltable_filename, implode("\n", $urltable)); + } } unlink_if_exists($tmp_urltable_filename); } else { -- cgit v1.1