[Mono-bugs] [Bug 60766][Maj] Changed - MOD_MONO & XSP both have fatal errors on OS X

[bugzilla-daemon@bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by grompf@sublimeintervention.com.

http://bugzilla.ximian.com/show_bug.cgi?id=60766

--- shadow/60766	2004-07-26 13:08:04.000000000 -0400
+++ shadow/60766.tmp.24237	2004-07-30 12:51:24.000000000 -0400
@@ -215,6 +215,39 @@
 to mask the issue.
 
 Additionally; the same problem is exhibited attempting to run the nunit tests.
 
 -kangaroo
 
+
+------- Additional Comments From grompf@sublimeintervention.com  2004-07-30 12:51 -------
+lupus,
+
+  Using the patch you suggested:
+=======================================================
+============
+RCS file: /cvs/public/mono/mono/mini/mini-ppc.c,v
+retrieving revision 1.73
+diff -u -r1.73 mini-ppc.c
+--- mini/mini-ppc.c     30 Jul 2004 12:49:43 -0000      1.73
++++ mini/mini-ppc.c     30 Jul 2004 15:37:10 -0000
+@@ -2810,7 +2810,7 @@
+                        int alloca_waste = PPC_STACK_PARAM_OFFSET + cfg->param_area + 31;
+                        int area_offset = alloca_waste;
+                        area_offset &= ~31;
+-                       ppc_addi (code, ppc_r11, ins->sreg1, alloca_waste + 31);
++                       ppc_addi (code, ppc_r11, ins->sreg1, alloca_waste + 63);
+                        ppc_rlwinm (code, ppc_r11, ppc_r11, 0, 0, 27);
+                        ppc_lwz (code, ppc_r0, 0, ppc_sp);
+                        ppc_neg (code, ppc_r11, ppc_r11);
+
+XSP gets a little farther; and throws a nullref internally on line 389 of 
+ModMonoWorkerRequest.cs; however this isn't a XSP bug (string[] headers is getting 
+b0rked somehow; it thinks its not null; but if you try to access it, it throws a nullrefex).
+
+Adding a try catch block around the GetRequestHeader method will catch the exception 
+and allow _some_ functionality; but all cookies are broken because all headers are broken.
+
+the same patch doesn not change the behaviour of nunit at all.
+
+-kangaroo
+