Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
secure the stack size parameter for thread,
authorAugustin Degomme <degomme@idpann.imag.fr>
Wed, 21 Nov 2012 16:23:18 +0000 (17:23 +0100)
committerAugustin Degomme <degomme@idpann.imag.fr>
Wed, 21 Nov 2012 17:15:10 +0000 (18:15 +0100)
to avoid some segfaults when the value is not right

src/xbt/xbt_os_thread.c

index d07112d..4b7d8fb 100644 (file)
@@ -194,8 +194,17 @@ xbt_os_thread_t xbt_os_thread_create(const char *name,
 
 void xbt_os_thread_setstacksize(int stack_size)
 {
 
 void xbt_os_thread_setstacksize(int stack_size)
 {
+  size_t def=0;
+  if(stack_size<0)xbt_die("stack size is negative, maybe it exceeds MAX_INT?\n");
   pthread_attr_init(&attr);
   pthread_attr_init(&attr);
-  pthread_attr_setstacksize (&attr, stack_size);
+  pthread_attr_getstacksize (&attr, &def);
+  int res = pthread_attr_setstacksize (&attr, stack_size);
+  if ( res!=0 ) {
+    if(res==EINVAL)XBT_WARN("Thread stack size is either < PTHREAD_STACK_MIN, > the max limit of the system, or perhaps not a multiple of PTHREAD_STACK_MIN - The parameter was ignored");
+    else XBT_WARN("unknown error in pthread stacksize setting");
+
+    pthread_attr_setstacksize (&attr, def);
+  }
   thread_attr_inited=1;
 }
 
   thread_attr_inited=1;
 }