[Mono-bugs] [Bug 52417][Min] New - unsafe possesion of a Monitor.

bugzilla-daemon@bugzilla.ximian.com bugzilla-daemon@bugzilla.ximian.com
Mon, 29 Mar 2004 08:04:01 -0500 (EST)


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 lluis@ximian.com.

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

--- shadow/52417	2004-03-29 08:04:01.000000000 -0500
+++ shadow/52417.tmp.24700	2004-03-29 08:04:01.000000000 -0500
@@ -0,0 +1,34 @@
+Bug#: 52417
+Product: Mono: Class Libraries
+Version: unspecified
+OS: All
+OS Details: 
+Status: RESOLVED   
+Resolution: FIXED
+Severity: Unknown
+Priority: Minor
+Component: System
+AssignedTo: mono-bugs@ximian.com                            
+ReportedBy: mordechait@mainsoft.com               
+QAContact: mono-bugs@ximian.com
+TargetMilestone: ---
+URL: 
+Cc: 
+Summary: unsafe possesion of a Monitor.
+
+Its not a problem but more of a programming paradigm when holding a Monitor
+(for example for a crtitical section) (thus i rate this bug minor).
+Where ?
+Module: System.dll 
+File:   System\Net\HttpWebRequest.cs line: 513 method: BeginGetRequestStream
+The programmer declares a Monitor on "this" object.
+A safe progammer would wrap this section with try - finally , and release
+the Monitor in the finally clause.
+Read more in MSDN for C#:
+ms-help://MS.VSCC.2003/MS.MSDNQTR.2003FEB.1033/cpref/html/frlrfsystemthreadingmonitorclasstopic.htm
+
+I take care for the quality of the code.
+Mr. Taitelman
+
+------- Additional Comments From lluis@ximian.com  2004-03-29 08:04 -------
+Fixed in CVS. Thanks for the suggestion!