Changeset 20087

Timestamp:

04/20/21 23:27:53 (4 years ago)

Author:

tbretz

Message:

Fixed an (un)signed comparison; in case of a bad file descriptor, it seems the connection terminated (looks like a featue of the most recent boost). trying to close the connection can produce crashes. So better avoid that.

File:

• trunk/FACT++/src/pwrctrl.cc (modified) (3 diffs)

trunk/FACT++/src/pwrctrl.cc

@@ -85 +85 @@
         const QDomNodeList imageElems = doc.elementsByTagName("span");
 
-        for (unsigned int i=0; i<imageElems.length(); i++)
+        for (int i=0; i<imageElems.length(); i++)
         {
             const QDomElement e = imageElems.item(i).toElement();
@@ -111 +111 @@
         if (bytes_received==0 || err)
         {
-            if (err==ba::error::eof)
+            if (err==ba::error::eof || err==ba::error::bad_descriptor)
             {
                 if (!fRdfData.empty())
@@ -120 +120 @@
             // 107: Transport endpoint is not connected (bs::error_code(107, bs::system_category))
             // 125: Operation canceled
-            if (err && err!=ba::error::eof &&                     // Connection closed by remote host
+            if (err && 
                 err!=ba::error::basic_errors::not_connected &&    // Connection closed by remote host
                 err!=ba::error::basic_errors::operation_aborted)  // Connection closed by us
             {
                 ostringstream str;
-                str << "Reading from " << URL() << ": " << err.message() << " (" << err << ")";// << endl;
+                str << "Reading[" << is_open() << "] from " << URL() << ": " << err.message() << " (" << err << ")";// << endl;
                 Error(str);
             }