libt2n: (tomj) added exception handling to every child after fork(). This is needed...
[libt2n] / test / timeout.cpp
index 7a4e9ed..3658d6d 100644 (file)
@@ -138,6 +138,7 @@ class test_timeout : public TestFixture
     CPPUNIT_TEST(CommandTimeout);
     CPPUNIT_TEST(CommandSlowResponse);
     CPPUNIT_TEST(DisconnectOnWrite);
+    CPPUNIT_TEST(WriteTwice);
     CPPUNIT_TEST(DisconnectOnRead);
     CPPUNIT_TEST(BreakAccept);
 
@@ -194,7 +195,13 @@ class test_timeout : public TestFixture
             case 0:
             // child
             {
-                socket_server ss("./socket");
+                try
+                {
+                    socket_server ss("./socket");
+                } catch(...)
+                {
+                    std::cerr << "exception in child. ignoring\n";
+                }
 
                 // don't call atexit and stuff
                 _exit(0);
@@ -231,11 +238,18 @@ class test_timeout : public TestFixture
             case 0:
             // child
             {
-                socket_server ss("./socket");
+                try
+                {
+                    socket_server ss("./socket");
+
+                    // max 10 sec
+                    for (int i=0; i < 10; i++)
+                        ss.fill_buffer(1000000);
+                } catch(...)
+                {
+                    std::cerr << "exception in child. ignoring\n";
+                }
 
-                // max 10 sec
-                for (int i=0; i < 10; i++)
-                    ss.fill_buffer(1000000);
                 // don't call atexit and stuff
                 _exit(0);
             }
@@ -273,24 +287,31 @@ class test_timeout : public TestFixture
             case 0:
             // child
             {
-                socket_server ss("./socket");
+                try
+                {
+                    socket_server ss("./socket");
 
-                // create a valid packet
-                ostringstream hello;
-                hello << "T2Nv" << PROTOCOL_VERSION << ';';
-                int byteordercheck=1;
-                hello.write((char*)&byteordercheck,sizeof(byteordercheck));
-                hello << ';';
+                    // create a valid packet
+                    ostringstream hello;
+                    hello << "T2Nv" << PROTOCOL_VERSION << ';';
+                    int byteordercheck=1;
+                    hello.write((char*)&byteordercheck,sizeof(byteordercheck));
+                    hello << ';';
 
-                packet_size_indicator psize=htonl(hello.str().size());
-                std::string send_data(hello.str());
-                send_data.insert(0,(char*)&psize,sizeof(packet_size_indicator));
+                    packet_size_indicator psize=htonl(hello.str().size());
+                    std::string send_data(hello.str());
+                    send_data.insert(0,(char*)&psize,sizeof(packet_size_indicator));
 
-                ss.add_callback(new_connection,bind(&test_timeout::send_slow_raw_socket, boost::ref(*this), send_data,&ss, _1));
+                    ss.add_callback(new_connection,bind(&test_timeout::send_slow_raw_socket, boost::ref(*this), send_data,&ss, _1));
+
+                    // max 10 sec
+                    for (int i=0; i < 10; i++)
+                        ss.fill_buffer(1000000);
+                } catch(...)
+                {
+                    std::cerr << "exception in child. ignoring\n";
+                }
 
-                // max 10 sec
-                for (int i=0; i < 10; i++)
-                    ss.fill_buffer(1000000);
                 // don't call atexit and stuff
                 _exit(0);
             }
@@ -328,19 +349,26 @@ class test_timeout : public TestFixture
             case 0:
             // child
             {
-                socket_server ss("./socket");
+                try
+                {
+                    socket_server ss("./socket");
 
-                ostringstream hello;
-                hello << "T2Nv" << PROTOCOL_VERSION << ';';
-                int byteordercheck=1;
-                hello.write((char*)&byteordercheck,sizeof(byteordercheck));
-                hello << ';';
+                    ostringstream hello;
+                    hello << "T2Nv" << PROTOCOL_VERSION << ';';
+                    int byteordercheck=1;
+                    hello.write((char*)&byteordercheck,sizeof(byteordercheck));
+                    hello << ';';
 
-                ss.add_callback(new_connection,bind(&test_timeout::send_hello, boost::ref(*this), hello.str(),&ss, _1));
+                    ss.add_callback(new_connection,bind(&test_timeout::send_hello, boost::ref(*this), hello.str(),&ss, _1));
+
+                    // max 10 sec
+                    for (int i=0; i < 10; i++)
+                        ss.fill_buffer(1000000);
+                } catch(...)
+                {
+                    std::cerr << "exception in child. ignoring\n";
+                }
 
-                // max 10 sec
-                for (int i=0; i < 10; i++)
-                    ss.fill_buffer(1000000);
                 // don't call atexit and stuff
                 _exit(0);
             }
@@ -385,34 +413,41 @@ class test_timeout : public TestFixture
             case 0:
             // child
             {
-                socket_server ss("./socket");
+                try
+                {
+                    socket_server ss("./socket");
 
-                ostringstream hello;
-                hello << "T2Nv" << PROTOCOL_VERSION << ';';
-                int byteordercheck=1;
-                hello.write((char*)&byteordercheck,sizeof(byteordercheck));
-                hello << ';';
+                    ostringstream hello;
+                    hello << "T2Nv" << PROTOCOL_VERSION << ';';
+                    int byteordercheck=1;
+                    hello.write((char*)&byteordercheck,sizeof(byteordercheck));
+                    hello << ';';
 
-                ss.add_callback(new_connection,bind(&test_timeout::send_hello, boost::ref(*this), hello.str(),&ss, _1));
+                    ss.add_callback(new_connection,bind(&test_timeout::send_hello, boost::ref(*this), hello.str(),&ss, _1));
 
-                // max 10 sec
-                for (int i=0; i < 10; i++)
-                {
-                    ss.fill_buffer(1000000);
+                    // max 10 sec
+                    for (int i=0; i < 10; i++)
+                    {
+                        ss.fill_buffer(1000000);
 
-                    string data;
-                    unsigned int cid;
+                        string data;
+                        unsigned int cid;
 
-                    if(ss.get_packet(data,cid))
-                    {
-                        // create a valid packet & send
-                        string response="abcdefghijklmnopqrstuvwxyz";
-                        packet_size_indicator psize=htonl(response.size());
-                        std::string send_data(response);
-                        send_data.insert(0,(char*)&psize,sizeof(packet_size_indicator));
-                        send_slow_raw_socket(send_data,&ss,cid);
+                        if(ss.get_packet(data,cid))
+                        {
+                            // create a valid packet & send
+                            string response="abcdefghijklmnopqrstuvwxyz";
+                            packet_size_indicator psize=htonl(response.size());
+                            std::string send_data(response);
+                            send_data.insert(0,(char*)&psize,sizeof(packet_size_indicator));
+                            send_slow_raw_socket(send_data,&ss,cid);
+                        }
                     }
+                } catch(...)
+                {
+                    std::cerr << "exception in child. ignoring\n";
                 }
+
                 // don't call atexit and stuff
                 _exit(0);
             }
@@ -457,10 +492,17 @@ class test_timeout : public TestFixture
             case 0:
             // child
             {
-                socket_server ss("./socket");
+                try
+                {
+                    socket_server ss("./socket");
+
+                    // bail out as soon as we get something
+                    ss.fill_buffer(-1);
+                } catch(...)
+                {
+                    std::cerr << "exception in child. ignoring\n";
+                }
 
-                // bail out as soon as we get something
-                ss.fill_buffer(-1);
                 // don't call atexit and stuff
                 _exit(0);
             }
@@ -495,6 +537,67 @@ class test_timeout : public TestFixture
         }
     }
 
+    void WriteTwice()
+    {
+        switch(child_pid=fork())
+        {
+            case -1:
+            {
+                CPPUNIT_FAIL("fork error");
+                break;
+            }
+            case 0:
+            // child
+            {
+                try
+                {
+                    socket_server ss("./socket");
+
+                    // bail out as soon as we get something
+                    ss.fill_buffer(-1);
+                    ss.fill_buffer(-1);
+                } catch(...)
+                {
+                    std::cerr << "exception in child. ignoring\n";
+                }
+
+                // don't call atexit and stuff
+                _exit(0);
+            }
+
+            default:
+            // parent
+            {
+                string data;
+
+                // don't kill us on broken pipe
+                signal(SIGPIPE, SIG_IGN);
+
+                // wait till server is up
+                sleep(1);
+                socket_client_connection sc("./socket");
+
+                string errormsg;
+
+                sc.write("somedata");
+
+                sleep(1);
+
+                // server should disconnect now
+                try
+                {
+                    sc.write("other data");
+                }
+                catch(t2n_transfer_error &e)
+                { errormsg=e.what(); }
+                catch(...)
+                { throw; }
+
+                CPPUNIT_ASSERT_EQUAL(string("write() returned Broken pipe"),errormsg);
+            }
+        }
+    }
+
     void DisconnectOnRead()
     {
         pid_t pid2;
@@ -509,14 +612,20 @@ class test_timeout : public TestFixture
             case 0:
             // child
             {
-                // wait till server is up
-                sleep(1);
+                try
+                {
+                    // wait till server is up
+                    sleep(1);
 
-                socket_client_connection sc("./socket");
+                    socket_client_connection sc("./socket");
 
-                // this is an evil hack to get access to real_write, don't ever do this in an app!!!
-                real_write_client_connection *rwc=(real_write_client_connection*)&sc;
-                rwc->real_write(string(10000,'x'));
+                    // this is an evil hack to get access to real_write, don't ever do this in an app!!!
+                    real_write_client_connection *rwc=(real_write_client_connection*)&sc;
+                    rwc->real_write(string(10000,'x'));
+                } catch(...)
+                {
+                    std::cerr << "exception in child. ignoring\n";
+                }
 
                 // don't call atexit and stuff
                 _exit(0);
@@ -553,10 +662,16 @@ class test_timeout : public TestFixture
                     case 0:
                     // child
                     {
-                        socket_client_connection *sc=new socket_client_connection("./socket");
-                        sc->write(string(10000,'x'));
-                        delete sc;
-                        // socket is closed regularly
+                        try
+                        {
+                            socket_client_connection *sc=new socket_client_connection("./socket");
+                            sc->write(string(10000,'x'));
+                            // socket is closed regularly
+                            delete sc;
+                        } catch(...)
+                        {
+                            std::cerr << "exception in child. ignoring\n";
+                        }
 
                         // don't run regular cleanup, otherwise cppunit stuff gets called
                         _exit(0);
@@ -600,11 +715,17 @@ class test_timeout : public TestFixture
             case 0:
             // child
             {
-                // wait till server is really up and waiting
-                sleep(2);
+                try
+                {
+                    // wait till server is really up and waiting
+                    sleep(2);
 
-                // connect with very tight timeout and only 1 retry
-                socket_client_connection sc("./socket",50,1);
+                    // connect with very tight timeout and only 1 retry
+                    socket_client_connection sc("./socket",50,1);
+                } catch(...)
+                {
+                    std::cerr << "exception in child. ignoring\n";
+                }
 
                 // don't call atexit and stuff
                 _exit(0);
@@ -644,10 +765,16 @@ class test_timeout : public TestFixture
                     case 0:
                     // child
                     {
-                        socket_client_connection *sc=new socket_client_connection("./socket");
-                        sc->write(string(10000,'x'));
-                        delete sc;
-                        // socket is closed regularly
+                        try
+                        {
+                            socket_client_connection *sc=new socket_client_connection("./socket");
+                            sc->write(string(10000,'x'));
+                            delete sc;
+                            // socket is closed regularly
+                        } catch(...)
+                        {
+                            std::cerr << "exception in child. ignoring\n";
+                        }
 
                         // don't run regular cleanup, otherwise cppunit stuff gets called
                         _exit(0);