[EMAIL PROTECTED] wrote:
Author: mturk
Date: Tue Oct 24 00:01:44 2006
New Revision: 467259
...
-    rc = ajp_msg_create(r->pool, AJP_HEADER_SZ_LEN+1, &msg);
+    rc = ajp_msg_create(r->pool, 128, &msg);

Please don't use an arbitrary number like this. Make a define, do something, anything but a bare 128 which someone coming along later will have no chance to understand.

Also, it looks like the message only contains a unit8. Why does this need 128 bytes?

Thanks,

-Paul

Reply via email to