Changeset 478 for trunk/wifidog/src/util.c

Timestamp:

02/24/05 23:03:38 (8 years ago)

Author:

minaguib

Message:

Better logging of details and calling of mark_* (auth+online/offline)

Files:

• trunk/wifidog/src/util.c (modified) (3 diffs)

trunk/wifidog/src/util.c

@@ -113 +113 @@
                 free(h_addr);
                 UNLOCK_GHBN();
-                mark_offline();
                 return NULL;
         }
@@ -156 +155 @@
 
 void mark_online() {
-        int isonline;
-        isonline = is_online();
+        int before;
+        int after;
+
+        before = is_online();
         time(&last_online_time);
-        if (!isonline) {
-                debug(LOG_INFO, "ONLINE status changed to ON");
-        }
+        after = is_online();
+
+        if (before != after) {
+                debug(LOG_INFO, "ONLINE status became %s", (after ? "ON" : "OFF"));
+        }
+
 }
 
 void mark_offline() {
-        int isonline;
-        isonline = is_online();
+        int before;
+        int after;
+
+        before = is_online();
         time(&last_offline_time);
+        after = is_online();
+
+        if (before != after) {
+                debug(LOG_INFO, "ONLINE status became %s", (after ? "ON" : "OFF"));
+        }
+
         /* If we're offline it definately means the auth server is offline */
         mark_auth_offline();
-        if (isonline) {
-                debug(LOG_INFO, "ONLINE status changed to OFF");
-        }
+
 }
 
@@ -187 +197 @@
 
 void mark_auth_online() {
-        int isauthonline;
-        isauthonline = is_auth_online();
+        int before;
+        int after;
+
+        before = is_auth_online();
         time(&last_auth_online_time);
+        after = is_auth_online();
+
+        if (before != after) {
+                debug(LOG_INFO, "AUTH_ONLINE status became %s", (after ? "ON" : "OFF"));
+        }
+
         /* If auth server is online it means we're definately online */
         mark_online();
-        if (!isauthonline) {
-                debug(LOG_INFO, "AUTH_ONLINE status changed to ON");
-        }
+
 }
 
 void mark_auth_offline() {
-        int isauthonline;
-        isauthonline = is_auth_online();
+        int before;
+        int after;
+
+        before = is_auth_online();
         time(&last_auth_offline_time);
-        if (isauthonline) {
-                debug(LOG_INFO, "AUTH_ONLINE status changed to OFF");
-        }
+        after = is_auth_online();
+
+        if (before != after) {
+                debug(LOG_INFO, "AUTH_ONLINE status became %s", (after ? "ON" : "OFF"));
+        }
+
 }