[asterisk-commits] wdoekes: branch 11 r424878 - in /branches/11: ./ contrib/scripts/safe_asterisk

[SVN commits to the Asterisk project]

Author: wdoekes
Date: Thu Oct  9 03:06:26 2014
New Revision: 424878

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=424878
Log:
safe_asterisk: Don't automatically exceed MAXFILES value of 2^20.

On systems with lots of RAM (e.g. 24GB) /proc/sys/fs/file-max divided
by two can exceed the per-process file limit of 2^20. This patch
ensures the value is capped.

(Patch cleaned up by me.)

ASTERISK-24011 #close
Reported by: Michael Myles
Patches:
  safe_asterisk-ulimit.diff uploaded by Michael Myles (License #6626)
........

Merged revisions 424875 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    branches/11/   (props changed)
    branches/11/contrib/scripts/safe_asterisk

Propchange: branches/11/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/11/contrib/scripts/safe_asterisk
URL: http://svnview.digium.com/svn/asterisk/branches/11/contrib/scripts/safe_asterisk?view=diff&rev=424878&r1=424877&r2=424878
==============================================================================
--- branches/11/contrib/scripts/safe_asterisk (original)
+++ branches/11/contrib/scripts/safe_asterisk Thu Oct  9 03:06:26 2014
@@ -66,12 +66,17 @@
 	message "safe_asterisk was started by `id -n` (uid `id -u`)."
 else
 	if `uname -s | grep Linux >/dev/null 2>&1`; then
-		# maximum number of open files is set to the system maximum divided by two if
-		# MAXFILES is not set.
+		# maximum number of open files is set to the system maximum
+		# divided by two if MAXFILES is not set.
 		if test -z "$MAXFILES"; then
 			# just check if file-max is readable
 			if test -r /proc/sys/fs/file-max; then
-				MAXFILES=$(( `cat /proc/sys/fs/file-max` / 2 ))
+				MAXFILES=$((`cat /proc/sys/fs/file-max` / 2))
+				# don't exceed upper limit of 2^20 for open
+				# files on systems where file-max is > 2^21
+				if test $MAXFILES -gt 1048576; then
+					MAXFILES=1048576
+				fi
 			fi
 		fi
 		SYSCTL_MAXFILES="fs.file-max"