Fix tshark.c to pass checkAPIs script

Since tshark.c was using strdup, perror, and g_main_quit, changes to
the file won't pass checkAPIs; so this commit replaces those with
the approved functions; except strdup, which was unecessary.

Change-Id: I031aa44594f2b96960a45f48537ab4e9a10d34b1
Reviewed-on: https://code.wireshark.org/review/898
Reviewed-by: Hadriel Kaplan <hadrielk@yahoo.com>
Reviewed-by: Anders Broman <a.broman58@gmail.com>

1 files changed, 3 insertions, 3 deletions

diff --git a/tshark.c b/tshark.c
index 2d5e557a43..4efa42279d 100644
--- a/tshark.c
+++ b/tshark.c
@@ -1477,7 +1477,7 @@ main(int argc, char *argv[])
         /* already processed; just ignore it now */
         break;
     case 'S':        /* Set the line Separator to be printed between packets */
-      separator = strdup(optarg);
+      separator = optarg;
       break;
     case 't':        /* Time stamp type */
       if (strcmp(optarg, "r") == 0)
@@ -2515,7 +2515,7 @@ capture(void)
 
       if (ret == -1)
       {
-        perror("select()");
+        fprintf(stderr, "%s: %s\n", "select()", g_strerror(errno));
         return TRUE;
       } else if (ret == 1) {
 #endif
@@ -2813,7 +2813,7 @@ capture_input_closed(capture_session *cap_session, gchar *msg)
   }
 #ifdef USE_BROKEN_G_MAIN_LOOP
   /*g_main_loop_quit(loop);*/
-  g_main_quit(loop);
+  g_main_loop_quit(loop);
 #else
   loop_running = FALSE;
 #endif