httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From j..@apache.org
Subject svn commit: r1030496 - /httpd/httpd/trunk/modules/session/mod_session.c
Date Wed, 03 Nov 2010 15:21:51 GMT
Author: jim
Date: Wed Nov  3 15:21:50 2010
New Revision: 1030496

URL: http://svn.apache.org/viewvc?rev=1030496&view=rev
Log:
Note for future dev....

Modified:
    httpd/httpd/trunk/modules/session/mod_session.c

Modified: httpd/httpd/trunk/modules/session/mod_session.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/session/mod_session.c?rev=1030496&r1=1030495&r2=1030496&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/session/mod_session.c (original)
+++ httpd/httpd/trunk/modules/session/mod_session.c Wed Nov  3 15:21:50 2010
@@ -89,6 +89,7 @@ static int session_included(request_rec 
  * @param r The request
  * @param z A pointer to where the session will be written.
  */
+/* ??? We return errors but we ignore them thru-out. ??? */
 static int ap_session_load(request_rec * r, session_rec ** z)
 {
 
@@ -168,6 +169,7 @@ static int ap_session_load(request_rec *
  * @param r The request
  * @param z A pointer to where the session will be written.
  */
+/* ??? We return errors but we ignore them thru-out. ??? */
 static int ap_session_save(request_rec * r, session_rec * z)
 {
     if (z) {
@@ -241,7 +243,7 @@ static int ap_session_save(request_rec *
 static void ap_session_get(request_rec * r, session_rec * z, const char *key, const char
**value)
 {
     if (!z) {
-        ap_session_load(r, &z);
+        ap_session_load(r, &z); /* errors ignored?? */
     }
     if (z && z->entries) {
         *value = apr_table_get(z->entries, key);
@@ -264,7 +266,7 @@ static void ap_session_set(request_rec *
                                 const char *key, const char *value)
 {
     if (!z) {
-        ap_session_load(r, &z);
+        ap_session_load(r, &z); /* errors ignored?? */
     }
     if (z) {
         if (value) {
@@ -420,7 +422,7 @@ static apr_status_t session_output_filte
                                                       &session_module);
 
         /* load the session, or create one if necessary */
-        ap_session_load(r, &z);
+        ap_session_load(r, &z); /* errors ignored?? */
         if (!z || z->written) {
             r = r->next;
             continue;
@@ -439,7 +441,7 @@ static apr_status_t session_output_filte
         }
 
         /* save away the session, and we're done */
-        ap_session_save(r, z);
+        ap_session_save(r, z); /* errors ignored?? */
 
         r = r->next;
     }
@@ -477,7 +479,7 @@ static int session_fixups(request_rec * 
                                                   &session_module);
 
     session_rec *z = NULL;
-    ap_session_load(r, &z);
+    ap_session_load(r, &z); /* errors ignored?? */
 
     if (conf->env) {
         session_identity_encode(r, z);



Mime
View raw message