testbot/testagentd: Reset the status when handling a new client.

Francois Gouget fgouget at codeweavers.com
Tue Dec 11 05:17:13 CST 2012


Otherwise, once the status has been set to FATAL, it cannot be set to 
anything else anymore, even for other clients.
---
 testbot/src/testagentd/testagentd.c |    6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/testbot/src/testagentd/testagentd.c b/testbot/src/testagentd/testagentd.c
index fc3a840..13d16e4 100644
--- a/testbot/src/testagentd/testagentd.c
+++ b/testbot/src/testagentd/testagentd.c
@@ -155,6 +155,8 @@ void set_status(int newstatus, const char* format, ...)
         status = newstatus;
         if (newstatus == ST_FATAL)
             broken = 1;
+        else if (newstatus == ST_OK)
+            broken = 0;
         va_start(valist, format);
         vset_status_msg(format, valist);
         va_end(valist);
@@ -1145,7 +1147,9 @@ int main(int argc, char** argv)
         {
             if (is_host_allowed(client, opt_srchost, addrlen))
             {
-                broken = 0;
+                /* Reset the status so new non-fatal errors can be set */
+                set_status(ST_OK, "ok");
+
                 /* Send the version right away */
                 send_string(client, PROTOCOL_VERSION);
 
-- 
1.7.10.4



More information about the wine-patches mailing list