Skip to content

Commit 1ac27e9

Browse files
committed
fixed pipe and zoombie problems
1.we don't need stdin here, and it'll cause OSError: too many openfiles problem. 2. after using /scan/taskid/stop , process turned into a zoombie, need add wait()
1 parent 62f3569 commit 1ac27e9

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

lib/utils/api.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -155,11 +155,12 @@ def reset_options(self):
155155

156156
def engine_start(self):
157157
self.process = Popen(["python", "sqlmap.py", "--pickled-options", base64pickle(self.options)],
158-
shell=False, stdin=PIPE, close_fds=not IS_WIN)
158+
shell=False, close_fds=not IS_WIN)
159159

160160
def engine_stop(self):
161161
if self.process:
162-
return self.process.terminate()
162+
self.process.terminate()
163+
return self.process.wait()
163164
else:
164165
return None
165166

@@ -168,7 +169,8 @@ def engine_process(self):
168169

169170
def engine_kill(self):
170171
if self.process:
171-
return self.process.kill()
172+
self.process.kill()
173+
return self.process.wait()
172174
else:
173175
return None
174176

0 commit comments

Comments
 (0)