Fix memory leak - make address sanitizer mandatory (icecc#226)

* fix memory leak on local build * make sanitizers mandatory
Kryndex · Apr 21, 2017 · 5e52a0b · 5e52a0b
1 parent f67bfa1
commit 5e52a0b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -35,11 +35,6 @@ matrix:
     before_install:
       - brew update
       - brew install lzo docbook2x
-  allow_failures:
-  - compiler: clang
-    env: BUILD_TYPE=asan
-  - compiler: clang
-    env: BUILD_TYPE=lsan
 
 before_script:
   - |

diff --git a/client/main.cpp b/client/main.cpp
@@ -488,10 +488,12 @@ int main(int argc, char **argv)
         /* If we can't talk to the daemon anymore we need to fall back
            to lock file locking.  */
         if (!startme || startme->type != M_JOB_LOCAL_BEGIN) {
+            delete startme;
             goto do_local_error;
         }
 
         ret = build_local(job, local_daemon, &ru);
+        delete startme;
     } else {
         try {
             // check if it should be compiled three times