[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!