lkml.org 
[lkml]   [1996]   [Jul]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectResource limit test
Date
Here's my patch to make binfmt_elf test the datasize before starting a
binary. It just returns ENOMEM for those binaries that would core dump
because of missing memory anyway. But I prefer to get an error message over
a core dump if possible.

Michael

--- /usr/src/linux/fs/binfmt_elf.c.old Wed Jun 12 10:09:48 1996
+++ /usr/src/linux/fs/binfmt_elf.c Mon Jul 8 20:32:06 1996
@@ -7,6 +7,7 @@
* Tools".
*
* Copyright 1993, 1994: Eric Youngdale (ericy@cais.com).
+ * Added resource limits check: Michael Meskes (meskes@informatik.rwth-aachen.de)
*/

#include <linux/module.h>
@@ -351,8 +352,9 @@
unsigned int elf_entry, interp_load_addr = 0;
int status;
unsigned int start_code, end_code, end_data;
- unsigned int elf_stack;
+ unsigned int elf_stack, datasize;
char passed_fileno[6];
+ unsigned long rlim;

ibcs2_interpreter = 0;
status = 0;
@@ -515,6 +517,24 @@
return -E2BIG;
}
}
+
+ /* MM: Check if we extend resource limits */
+ /* Algorithm proposed by Richard Henderson <rth@tamu.edu> */
+ rlim = current->rlim[RLIMIT_DATA].rlim_cur;
+ if (rlim >= RLIM_INFINITY)
+ rlim = ~0;
+ elf_ppnt = elf_phdata;
+ datasize = 0;
+ for(i=0;i < elf_ex.e_phnum; i++){
+ if(elf_ppnt->p_flags & PF_W) {
+ unsigned long new_data_size = datasize + elf_ppnt->p_memsz;
+ if (new_data_size > rlim || new_data_size < datasize)
+ return -ENOMEM;
+ datasize = new_data_size;
+ }
+ elf_ppnt++;
+ }
+ /* MM: End */

/* OK, This is the point of no return */
flush_old_exec(bprm);
--
Michael Meskes | _____ ________ __ ____
meskes@informatik.rwth-aachen.de | / ___// ____/ // / / __ \___ __________
meskes@sanet.de | \__ \/ /_ / // /_/ /_/ / _ \/ ___/ ___/
meskes@debian.org | ___/ / __/ /__ __/\__, / __/ / (__ )
Use Debian Linux! | /____/_/ /_/ /____/\___/_/ /____/

\
 
 \ /
  Last update: 2005-03-22 13:37    [W:0.057 / U:5.348 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site