2009-09-19 Ivan Maidanski <ivmai@mail.ru>

	* os_dep.c (GC_get_stack_base): Add FIXME; add assertion for
	GC_get_writable_length() result.

diff --git a/ChangeLog b/ChangeLog
index e886f33f16eee3d3b51e5d6eb03dee42099f6e5b..6fe02981f53714609d18b77f7218991693884c6d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2009-09-19  Ivan Maidanski <ivmai@mail.ru>
+
+       * os_dep.c (GC_get_stack_base): Add FIXME; add assertion for
+       GC_get_writable_length() result.
 
 2009-09-18  Ivan Maidanski <ivmai@mail.ru> (really Ludovic CourtXs)
 

diff --git a/os_dep.c b/os_dep.c
index d9123944d00affe6f0cb2ef8dc3189129f472d45..180ba1355f629baa03601ba7ffdc44a04ef19bcb 100644 (file)
--- a/os_dep.c
+++ b/os_dep.c
@@ -656,8 +656,10 @@ GC_API int GC_CALL GC_get_stack_base(struct GC_stack_base *sb)
     int dummy;
     ptr_t sp = (ptr_t)(&dummy);
     ptr_t trunc_sp = (ptr_t)((word)sp & ~(GC_page_size - 1));
+    /* FIXME: This won't work if called from a deeply recursive */
+    /* client code (and the committed stack space has grown).   */
     word size = GC_get_writable_length(trunc_sp, 0);
-   
+    GC_ASSERT(size != 0);
     sb -> mem_base = trunc_sp + size;
     return GC_SUCCESS;
 }