--- parser3/src/classes/form.C	2001/04/28 13:24:57	1.7
+++ parser3/src/classes/form.C	2002/08/01 11:41:12	1.28
@@ -1,23 +1,22 @@
 /** @file
 	Parser: @b form parser class.
 
-	Copyright (c) 2001 ArtLebedev Group (http://www.artlebedev.com)
-
-	Author: Alexander Petrosyan <paf@design.ru> (http://design.ru/paf)
-
-	$Id: form.C,v 1.7 2001/04/28 13:24:57 paf Exp $
+	Copyright (c) 2001, 2002 ArtLebedev Group (http://www.artlebedev.com)
+	Author: Alexandr Petrosian <paf@design.ru> (http://paf.design.ru)
 */
 
+static const char* IDENT_FORM_C="$Date: 2002/08/01 11:41:12 $";
+
 #include "classes.h"
 #include "pa_request.h"
 #include "pa_vform.h"
 
 /// $LIMITS.max_post_size default 10M
-const size_t MAX_POST_SIZE_DEFAULT=10*0x400*400;
+const size_t MAX_POST_SIZE_DEFAULT=10*0x400*0x400;
 
 // defines
 
-#define FORM_CLASS_NAME "form"
+#define LIMITS_NAME "LIMITS"
 
 #define MAX_POST_SIZE_NAME "post_max_size"
 
@@ -26,27 +25,27 @@ const size_t MAX_POST_SIZE_DEFAULT=10*0x
 class MForm : public Methoded {
 public:
 	MForm(Pool& pool);
-protected: // Methoded
+public: // Methoded
 	bool used_directly() { return false; }
 	void configure_admin(Request& r);
 private:
 	String max_post_size_name;
+	String limits_name;
 };
 
 // methods
 
 // constructor & configurator
 
-MForm::MForm(Pool& apool) : Methoded(apool),
-	max_post_size_name(apool, MAX_POST_SIZE_NAME)
-{
-	set_name(*NEW String(pool(), FORM_CLASS_NAME));
+MForm::MForm(Pool& apool) : Methoded(apool, "form"),
+	max_post_size_name(apool, MAX_POST_SIZE_NAME),
+	limits_name(apool, LIMITS_NAME) {
 }
 
 void MForm::configure_admin(Request& r) {
 	Pool& pool=r.pool();
 
-	Value *limits=r.main_class?r.main_class->get_element(*limits_name):0;
+	Value *limits=r.main_class?r.main_class->get_element(limits_name):0;
 	if(r.info.method && StrEqNc(r.info.method, "post", true)) {
 		// $limits.max_post_size default 10M
 		Value *element=limits?limits->get_element(max_post_size_name):0;
@@ -54,18 +53,25 @@ void MForm::configure_admin(Request& r)
 		size_t max_post_size=value?value:MAX_POST_SIZE_DEFAULT;
 		
 		if(r.info.content_length>max_post_size)
-			PTHROW(0, 0,
+			throw Exception("parser.runtime",
 				0,
 				"posted content_length(%u) > max_post_size(%u)",
-					r.post_size, max_post_size);
+					r.info.content_length, max_post_size);
+		if(r.info.content_length<0)
+			throw Exception(0,
+				0,
+				"posted content_length(%u) < 0",
+					r.info.content_length);
 
 		// read POST data
-		r.post_data=(char *)malloc(r.info.content_length);
-		r.post_size=SAPI::read_post(pool, r.post_data, r.info.content_length);
+		if(r.info.content_length) {
+			r.post_data=(char *)pool.malloc(r.info.content_length);
+			r.post_size=SAPI::read_post(pool, r.post_data, r.info.content_length);
+		}
 		if(r.post_size!=r.info.content_length)
-			PTHROW(0, 0, 
+			throw Exception(0, 
 				0, 
-				"post_size(%u)!=content_length(%u)", 
+				"post_size(%u) != content_length(%u)", 
 					r.post_size, r.info.content_length);
 	}
 }