Merge pull request #924 from doktornotor/patch-13

3 files changed, 44 insertions, 45 deletions

diff --git a/config/nmap/nmap.inc b/config/nmap/nmap.inc
index 632d4b2d..cbf1b15a 100644
--- a/config/nmap/nmap.inc
+++ b/config/nmap/nmap.inc
@@ -1,9 +1,9 @@
 <?
-/* $Id$ */
 /*
-	part of pfSense (https://www.pfsense.org/)
-
-	Copyright (C) 2006 Bill Marquette - bill.marquette@gmail.com.
+	nmap.inc
+	part of pfSense (https://www.pfSense.org/)
+	Copyright (C) 2006 Bill Marquette <bill.marquette@gmail.com>
+	Copyright (C) 2015 ESF, LLC
 	All rights reserved.
 
 	Redistribution and use in source and binary forms, with or without
@@ -43,20 +43,16 @@ function nmap_install() {
 	}
 	/* Only copy the file if it doesn't exist */
 	if (file_exists($source_file) && !file_exists($destination_file)) {
-		if (!is_dir(dirname($destination_file))) {
-			@mkdir(dirname($destination_file), 0644, true);
-		}
+		safe_mkdir(dirname($destination_file));
 		@symlink($source_file, $destination_file);
 	}
 }
 
-function nmap_custom_php_validation_command($post, & $input_errors) {
+function nmap_custom_php_validation_command($post, &$input_errors) {
 	global $_POST, $savemsg, $config;
 	if (empty($_POST['hostname'])) {
 		$input_errors[] = gettext("You must enter an IP address to scan.");
-	} elseif (!(is_ipaddr($_POST['hostname']) ||
-			is_subnet($_POST['hostname']) ||
-			is_hostname($_POST['hostname']))) {
+	} elseif (!(is_ipaddr($_POST['hostname']) || is_subnet($_POST['hostname']) || is_hostname($_POST['hostname']))) {
 		$input_errors[] = gettext("You must enter a valid IP address to scan.");
 	}
 
@@ -71,9 +67,9 @@ function nmap_custom_php_validation_command($post, & $input_errors) {
 function nmap_custom_add_php_command() {
 	$nmap_options = "";
 
-	if (function_exists("is_ipaddrv6") && function_exists("is_subnetv6"))
-		if (is_ipaddrv6($_POST['hostname']) || is_subnetv6($_POST['hostname']))
-			$nmap_options .= " -6";
+	if (is_ipaddrv6($_POST['hostname']) || is_subnetv6($_POST['hostname'])) {
+		$nmap_options .= " -6";
+	}
 
 	switch($_POST['scanmethod']) {
 		case 'syn':
@@ -93,14 +89,23 @@ function nmap_custom_add_php_command() {
 			break;
 	}
 
-	if($_POST['noping']) $nmap_options .= " -P0";
-	if($_POST['servicever']) $nmap_options .= " -sV";
-	if($_POST['osdetect']) $nmap_options .= " -O";
-	if(!empty($_POST['interface'])) $nmap_options .= " -e " . get_real_interface($_POST['interface']);
+	if ($_POST['noping']) {
+		$nmap_options .= " -P0";
+	}
+	if ($_POST['servicever']) {
+		$nmap_options .= " -sV";
+	}
+	if ($_POST['osdetect']) {
+		$nmap_options .= " -O";
+	}
+	if (!empty($_POST['interface'])) {
+		$nmap_options .= " -e " . get_real_interface($_POST['interface']);
+	}
 
 	$nmap_options .= " " . escapeshellarg($_POST['hostname']);
-	echo "Running: /usr/local/bin/nmap {$nmap_options}</br>";
+	echo "<strong>Running: /usr/local/bin/nmap {$nmap_options}</strong><br />";
 	system("/usr/local/bin/nmap" . $nmap_options);
+	echo "<br /><br /><input class=\"formbtn\" type=\"button\" value=\"Back to NMap\" onclick=\"history.back()\" />";
 }
 
 function nmap_get_interfaces() {
diff --git a/config/nmap/nmap.xml b/config/nmap/nmap.xml
index ad0f8e0a..446ba461 100644
--- a/config/nmap/nmap.xml
+++ b/config/nmap/nmap.xml
@@ -7,26 +7,25 @@
 /* $Id$ */
 /* ========================================================================== */
 /*
-	authng.xml
-	part of pfSense (http://www.pfSense.com)
-	Copyright (C) 2007 to whom it may belong
-	All rights reserved.
-
-	Based on m0n0wall (http://m0n0.ch/wall)
-	Copyright (C) 2003-2006 Manuel Kasper <mk@neon1.net>.
+	nmap.xml
+	part of pfSense (https://www.pfSense.org/)
+	Copyright (C) 2006 Bill Marquette <bill.marquette@gmail.com>
+	Copyright (C) 2015 ESF, LLC
 	All rights reserved.
 */
-/* ========================================================================== */
+/* ====================================================================================== */
 /*
 	Redistribution and use in source and binary forms, with or without
 	modification, are permitted provided that the following conditions are met:
 
-	 1. Redistributions of source code must retain the above copyright notice,
-	this list of conditions and the following disclaimer.
 
-	 2. Redistributions in binary form must reproduce the above copyright
-	notice, this list of conditions and the following disclaimer in the
-	documentation and/or other materials provided with the distribution.
+	1. Redistributions of source code must retain the above copyright notice,
+	   this list of conditions and the following disclaimer.
+
+	2. Redistributions in binary form must reproduce the above copyright
+	   notice, this list of conditions and the following disclaimer in the
+	   documentation and/or other materials provided with the distribution.
+
 
 	THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES,
 	INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY
@@ -39,20 +38,16 @@
 	ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 	POSSIBILITY OF SUCH DAMAGE.
 */
-/* ========================================================================== */
+/* ====================================================================================== */
 ]]>
 	</copyright>
-	<description>Describe your package here</description>
-	<requirements>Describe your package requirements here</requirements>
-	<faq>Currently there are no FAQ items provided.</faq>
 	<name>nmap</name>
-	<version>6.40_2 pkg v1.3</version>
+	<version>1.4</version>
 	<title>Diagnostics: NMap</title>
 	<savetext>Scan</savetext>
 	<preoutput>yes</preoutput>
 	<donotsave>true</donotsave>
 	<include_file>/usr/local/pkg/nmap.inc</include_file>
-	<!-- Menu is where this packages menu will appear -->
 	<menu>
 		<name>NMap</name>
 		<tooltiptext>NMap is a utility for network exploration or security auditing. It supports ping scanning (determine which hosts are up), many port scanning techniques (determine what services the hosts are offering), version detection (determine what application/service is runing on a port), and TCP/IP fingerprinting (remote host OS or device identification). It also offers flexible target and port specification, decoy/stealth scanning, SunRPC scanning, and more. Most Unix and Windows platforms are supported in both GUI and command line modes. Several popular handheld devices are also supported, including the Sharp Zaurus and the iPAQ.</tooltiptext>
@@ -61,7 +56,6 @@
 	</menu>
 	<additional_files_needed>
 		<prefix>/usr/local/pkg/</prefix>
-		<chmod>077</chmod>
 		<item>https://packages.pfsense.org/packages/config/nmap/nmap.inc</item>
 	</additional_files_needed>
 	<fields>
@@ -97,23 +91,23 @@
 		<field>
 			<fielddescr>-P0</fielddescr>
 			<fieldname>noping</fieldname>
-			<description>This allows the scanning of networks that don't allow ICMP echo requests (or responses) through their firewall.  microsoft.com is an example of such a network, and thus you should always use -P0 or -PT80 when port scanning microsoft.com.  Note the "ping" in this context may involve more than the traditional ICMP echo request packet.  Nmap supports many such probes, including arbitrary combinations of TCP, UDP, and ICMP probes.  By default, Nmap sends an ICMP echo request and a TCP ACK packet to port 80.</description>
+			<description>This allows the scanning of networks that don't allow ICMP echo requests (or responses) through their firewall. microsoft.com is an example of such a network, and thus you should always use -P0 or -PT80 when port scanning microsoft.com. Note the "ping" in this context may involve more than the traditional ICMP echo request packet. Nmap supports many such probes, including arbitrary combinations of TCP, UDP, and ICMP probes. By default, Nmap sends an ICMP echo request and a TCP ACK packet to port 80.</description>
 			<type>checkbox</type>
 			<typehint>Do not try to ping hosts at all before scanning them.</typehint>
 		</field>
 		<field>
 			<fielddescr>-sV</fielddescr>
 			<fieldname>servicever</fieldname>
-			<description>After TCP and/or UDP ports are discovered using one of the other scan methods, version detection communicates with those ports to try and determine more about what is actually running.  A file called nmap-service-probes is used to determine the best probes for detecting various services and the match strings to expect.  Nmap tries to determine the service protocol (e.g. ftp, ssh, telnet, http), the application name (e.g. ISC Bind, Apache httpd, Solaris telnetd), the version number, and sometimes miscellaneous details like whether an X server is open to connections or the SSH protocol version)</description>
+			<description>After TCP and/or UDP ports are discovered using one of the other scan methods, version detection communicates with those ports to try and determine more about what is actually running. A file called nmap-service-probes is used to determine the best probes for detecting various services and the match strings to expect. Nmap tries to determine the service protocol (e.g. ftp, ssh, telnet, http), the application name (e.g. ISC Bind, Apache httpd, Solaris telnetd), the version number, and sometimes miscellaneous details like whether an X server is open to connections or the SSH protocol version).</description>
 			<type>checkbox</type>
-			<typehint>Try to identify service versions</typehint>
+			<typehint>Try to identify service versions.</typehint>
 		</field>
 		<field>
 			<fielddescr>-O</fielddescr>
 			<fieldname>osdetect</fieldname>
-			<description>This option activates remote host identification via TCP/IP fingerprinting.  In other words, it uses a bunch of techniques to detect subtleties in the underlying operating system network stack of the computers you are scanning.  It uses this information to create a "fingerprint" which it compares with its database of known OS fingerprints (the nmap-os-fingerprints file) to decide what type of system you are scanning</description>
+			<description>This option activates remote host identification via TCP/IP fingerprinting. In other words, it uses a bunch of techniques to detect subtleties in the underlying operating system network stack of the computers you are scanning. It uses this information to create a "fingerprint" which it compares with its database of known OS fingerprints (the nmap-os-fingerprints file) to decide what type of system you are scanning.</description>
 			<type>checkbox</type>
-			<typehint>Turn on OS detection</typehint>
+			<typehint>Turn on OS detection.</typehint>
 		</field>
 	</fields>
 	<custom_add_php_command>
diff --git a/pkg_config.10.xml b/pkg_config.10.xml
index 0a5173a5..56c3367c 100644
--- a/pkg_config.10.xml
+++ b/pkg_config.10.xml
@@ -759,7 +759,7 @@
 		<category>Security</category>
 		<depends_on_package_pbi>nmap-6.47-##ARCH##.pbi</depends_on_package_pbi>
 		<config_file>https://packages.pfsense.org/packages/config/nmap/nmap.xml</config_file>
-		<version>1.3</version>
+		<version>1.4</version>
 		<status>Stable</status>
 		<pkginfolink>https://doc.pfsense.org/index.php/Nmap_package</pkginfolink>
 		<required_version>2.2</required_version>