Revert all similar lint fixes

google · Jan 9, 2023 · e98c11a · e98c11a
1 parent 94cc54c
commit e98c11a
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 13 deletions.
diff --git a/fuzzers/eclipser/fuzzer.py b/fuzzers/eclipser/fuzzer.py
@@ -94,8 +94,8 @@ def eclipser(input_corpus, output_corpus, target_binary):
     if os.listdir(input_corpus):  # Specify inputs only if any seed exists.
         command += ['-i', input_corpus]
     print('[eclipser] Run Eclipser with command: ' + ' '.join(command))
-    with subprocess.Popen(command):
-        pass
+    # pylint: disable=consider-using-with
+    subprocess.Popen(command)
 
 
 def afl_worker(input_corpus, output_corpus, target_binary):

diff --git a/fuzzers/eclipser_aflplusplus/fuzzer.py b/fuzzers/eclipser_aflplusplus/fuzzer.py
@@ -82,8 +82,8 @@ def eclipser(input_corpus, output_corpus, target_binary):
     if os.listdir(input_corpus):  # Specify inputs only if any seed exists.
         command += ['-i', input_corpus]
     print('[eclipser] Run Eclipser with command: ' + ' '.join(command))
-    with subprocess.Popen(command):
-        pass
+    # pylint: disable=consider-using-with
+    subprocess.Popen(command)
 
 
 def afl_worker(input_corpus, output_corpus, target_binary):

diff --git a/fuzzers/fuzzolic_aflplusplus_fuzzy/fuzzer.py b/fuzzers/fuzzolic_aflplusplus_fuzzy/fuzzer.py
@@ -99,8 +99,8 @@ def fuzzolic(input_corpus, output_corpus, target_binary):
         target_binary,
     ]
     print('[fuzzolic] Running Fuzzolic with command: ' + ' '.join(command))
-    with subprocess.Popen(command):
-        pass
+    # pylint: disable=consider-using-with
+    subprocess.Popen(command)
 
 
 def afl_worker(input_corpus, output_corpus, target_binary):

diff --git a/fuzzers/fuzzolic_aflplusplus_z3/fuzzer.py b/fuzzers/fuzzolic_aflplusplus_z3/fuzzer.py
@@ -98,8 +98,8 @@ def fuzzolic(input_corpus, output_corpus, target_binary):
         target_binary,
     ]
     print('[fuzzolic] Running Fuzzolic with command: ' + ' '.join(command))
-    with subprocess.Popen(command):
-        pass
+    # pylint: disable=consider-using-with
+    subprocess.Popen(command)
 
 
 def afl_worker(input_corpus, output_corpus, target_binary):

diff --git a/fuzzers/neuzz/fuzzer.py b/fuzzers/neuzz/fuzzer.py
@@ -91,8 +91,8 @@ def run_neuzz(input_corpus,
         'python2', './nn.py', '--output-folder', afl_output_dir, target_binary
     ]
     print('[run_neuzz] Running command: ' + ' '.join(command))
-    with subprocess.Popen(command, stdout=output_stream, stderr=output_stream):
-        pass
+    # pylint: disable=consider-using-with
+    subprocess.Popen(command, stdout=output_stream, stderr=output_stream)
     time.sleep(40)
     target_rel_path = os.path.relpath(target_binary, os.getcwd())
     # Spinning up neuzz
@@ -101,9 +101,11 @@ def run_neuzz(input_corpus,
         target_rel_path, '@@'
     ]
     print('[run_neuzz] Running command: ' + ' '.join(command))
-    with subprocess.Popen(command, stdout=output_stream,
-                          stderr=output_stream) as neuzz_proc:
-        neuzz_proc.wait()
+    # pylint: disable=consider-using-with
+    neuzz_proc = subprocess.Popen(command,
+                                  stdout=output_stream,
+                                  stderr=output_stream)
+    neuzz_proc.wait()
 
 
 def fuzz(input_corpus, output_corpus, target_binary):