[Mono-bugs] [Bug 79701][Maj] New - memory leak with mod_mono

[bugzilla-daemon at 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 colin at univ-metz.fr.

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

--- shadow/79701	2006-10-19 15:44:26.000000000 -0400
+++ shadow/79701.tmp.11660	2006-10-19 15:44:26.000000000 -0400
@@ -0,0 +1,47 @@
+Bug#: 79701
+Product: Mono: Class Libraries
+Version: unspecified
+OS: 
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: 
+Priority: Major
+Component: Sys.Web
+AssignedTo: gonzalo at ximian.com                            
+ReportedBy: colin at univ-metz.fr               
+QAContact: mono-bugs at ximian.com
+TargetMilestone: ---
+URL: 
+Cc: 
+Summary: memory leak with mod_mono
+
+Hi gonzalo,
+
+there's some memory leak with mod_mono, when uploading large data using
+SEND_FROM_MEMORY command. read_data_string function is called many times ,
+and each time apr_pcalloc is called to get memory that is never released.
+I supposed you use the pool for other variable, so may be a solution will
+be to create a sub pool before calling read_data_string(...).
+
+apr_pool_t *temp_pool;
+
+....
+
+    case SEND_FROM_MEMORY:
+        apr_pool_create (&temp_pool,r->pool);
+        if (read_data_string (temp_pool, sock, &str, &size) == NULL) {
+            status = -1;
+            apr_pool_destroy(temp_pool);
+            break;
+        }
+        request_send_response_from_memory (r, str, size);
+        apr_pool_destroy(temp_pool);
+        break;
+
+probably this problem occure whith other command.
+
+
+cheers,
+
+cyrille.