Only stop PCAP dump after capture

This ensures that all the monitored data is actually exported, avoiding
a possible inconsistency between the data shown on the screen and the
exported PCAP data.

app/src/main/java/com/emanuelef/remote_capture/CaptureService.java

@@ -878,9 +878,7 @@ public static void stopService() {
             return;
 
         captureService.mStopping = true;
-
         stopPacketLoop();
-        captureService.signalServicesTermination();
 
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N)
             captureService.stopForeground(STOP_FOREGROUND_REMOVE);

app/src/main/jni/core/capture_libpcap.c

@@ -533,6 +533,9 @@ static void process_pcapd_rv(pcapdroid_t *pd, int rv) {
         case PCAPD_PCAP_READ_ERROR:
             log_f(PD_ERR_PCAP_READ);
             break;
+        case PCAPD_SOCKET_WRITE_ERROR:
+            // ignore, as it can be caused by PCAPdroid stopping the capture
+            break;
         default:
             log_f("pcapd daemon exited with code %d", rv);
     }