From a0a9c07027eacddd1a9568a477f2a797a846edef Mon Sep 17 00:00:00 2001
From: Xavier Roche <xroche@users.noreply.github.com>
Date: Thu, 15 Aug 2013 07:46:20 +0000
Subject: Removed unused and quite dangerous option.

---
 src/htscoremain.c | 19 +++++--------------
 1 file changed, 5 insertions(+), 14 deletions(-)

(limited to 'src/htscoremain.c')

diff --git a/src/htscoremain.c b/src/htscoremain.c
index 844c97f..b91f54c 100644
--- a/src/htscoremain.c
+++ b/src/htscoremain.c
@@ -2182,6 +2182,8 @@ HTSEXT_API int hts_main2(int argc, char **argv, httrackp * opt) {
                 return 0;
                 break;
               case '~':        /* internal lib test */
+                HTS_PANIC_PRINTF
+                  ("Option #~ is disabled for security reasons");
                 //Disabled because choke on GCC 4.3 (toni from links2linux.de)
                 //{
                 //  char thisIsATestYouShouldSeeAnError[12];
@@ -2547,21 +2549,10 @@ HTSEXT_API int hts_main2(int argc, char **argv, httrackp * opt) {
                 return 0;
                 break;
               case '!':
-                if (na + 1 >= argc) {
-                  HTS_PANIC_PRINTF
-                    ("Option #! needs to be followed by a commandline");
-                  printf("Example: '-#!' 'echo hello'\n");
-                  htsmain_free();
-                  return -1;
-                } else {
-                  int code;
-
-                  if ((code = system(argv[na + 1])) != 0) {
-                    fprintf(stderr, "process returned error code %d\n", code);
-                  }
-                }
+                HTS_PANIC_PRINTF
+                  ("Option #! is disabled for security reasons");
                 htsmain_free();
-                return 0;
+                return -1;
                 break;
               case 'd':
                 opt->parsedebug = 1;
-- 
cgit v1.2.3