From 6a71d251cd80b5c06267003942447c42faff697c Mon Sep 17 00:00:00 2001
From: idbeta <idbeta@gmail.com>
Date: Tue, 23 Feb 2021 13:31:01 +0800
Subject: [PATCH 1/4] test: add fuzzing case

---
 .github/workflows/fuzzing-ci.yaml             |   8 +-
 t/fuzzing/public.pyc                          | Bin 0 -> 1743 bytes
 t/fuzzing/serverless_route_test.py            | 112 ++++++++++++++++++
 .../{route_test.py => simpleroute_test.py}    |   0
 4 files changed, 118 insertions(+), 2 deletions(-)
 create mode 100644 t/fuzzing/public.pyc
 create mode 100644 t/fuzzing/serverless_route_test.py
 rename t/fuzzing/{route_test.py => simpleroute_test.py} (100%)

diff --git a/.github/workflows/fuzzing-ci.yaml b/.github/workflows/fuzzing-ci.yaml
index 1b27660c4ee3..634b8fb2e612 100644
--- a/.github/workflows/fuzzing-ci.yaml
+++ b/.github/workflows/fuzzing-ci.yaml
@@ -51,9 +51,13 @@ jobs:
       run: |
         pip install typing boofuzz
 
-    - name: run test
+    - name: run simpleroute test
       run: |
-        python $PWD/t/fuzzing/route_test.py
+        python $PWD/t/fuzzing/simpleroute_test.py
+    
+    - name: run serverless route test
+      run: |
+        python $PWD/t/fuzzing/serverless_route_test.py
 
     - name: echo test log
       # more than 150k lines, so only display the last log, actually, this step is not needed.
diff --git a/t/fuzzing/public.pyc b/t/fuzzing/public.pyc
new file mode 100644
index 0000000000000000000000000000000000000000..5d8b6a0677579d07301a2b74cadd1c1f77a791ce
GIT binary patch
literal 1743
zcmbtUQEwYX5T3oW-PlcN1t<_Iq$5fd3n_7g#9LK@XjSnNMQ0EwQe`f88+$Kj-}QFg
zrp10CNIb*;<5%zl;QRLM<ON=c&pY?+?9ATm%r_hUwbj#qMn9#p`UQ9&W7uyXGW?Yk
zB4;;P$(dT~133#M3*=WAgp!3Gv?ObJ(5COO-^M=XZ~RVfz7ayj?QI0%L~<p0um+*H
zmRu|T;lDHNGolj%A$~8%d1+xJQYCKU_yJoL#<>lkS|^29b21uTm_m<6ePFp2_7Ky-
z_g2?-V6t=VlEL)t*tkJb8EdWvZg4Sw_s-15gKB<WnDnrE>+ZoNo!6tx)E@y_Mj^@{
ziAx+WFzjy-3!k=~rSd3{on_!r2!uSfWT9j`Ys(_Qt0Rk0w#T6?TXOY67A<-9@+(=k
zSNT{VjrRnH&NjRzL}G%xC3#z}MV9dHNY<6hK<u%&j{KtJ)v^45$_@KH#L{HWC_uxQ
zEu&8p7d^JoA{y7aiVl)!_aZUHuKoC5*K}Q%^?xl!UmhJryG;=$x+o9=KH{=+?oe4|
zotbfr#LmyFx=gjT-ukqx^vqZByLzrYIQ{nYdF<_3TvRf1?*V*mQ7LE2*>eWcY)FHm
z99wrE;JhqZun`pQ0p?8^M?P-q9^UGblmSr59nxuL9oY!rn4<b_B<XUiiB2yW$|smu
z#xJVJCwQc`)P6%5a+U=3UGO^qsJ>kH0_oGV1FDX8K!X6Z1d(2=kgP(#phvJVGMRS-
z<fLjt@-DS`kFseYGB1$)zJPA@7exkk*r1@@GZ=w(;9S1x?Y8KqFdzCfpV%J}TV<mI
zeX(ZCx;5sNbtutvFdLiM)nQuBZcL_$b-G?V9K7~4TSUp5%V_`Ssx~th?R~M_UsDBj
z@>P#X9mgEj8m(r{Yn`$vq-}^X#IQRM6W?MlAIK^fB7X$(s)L}hD1G|)s#YfRw}-zn
z4v&0=t2<TLGP);LyR@8EPd~@RUj$w0v(PbDV{$#-q#!pv&)y`sop`#+&YdazWpcXe
z1)dBUX)>@y4}YP#+Eo5RODCqtYP1RslPrsw0e=(59|FaX=;HTIXcd3PIh$zJY<D`Z
zV?Yg~SKIADVcAfkI$(28?Wy<GLp*)P&}4wC@uw6#O1w?+7~(b*kN(d!yn$_=Ov`Lu
S=%*h5TV{Jx;R(87xAzYP%WDe&

literal 0
HcmV?d00001

diff --git a/t/fuzzing/serverless_route_test.py b/t/fuzzing/serverless_route_test.py
new file mode 100644
index 000000000000..190ce365e219
--- /dev/null
+++ b/t/fuzzing/serverless_route_test.py
@@ -0,0 +1,112 @@
+#! /usr/bin/env python
+
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+import subprocess
+from public import *
+from boofuzz import *
+
+def create_route():
+    command = '''curl -i http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1' -X PUT -d '
+{
+    "uri": "/post*",
+    "methods": ["POST"],
+    "plugins": {
+        "serverless-post-function": {
+            "functions": ["return function()\n local core = require(\"apisix.core\")\n   ngx.req.read_body()\n    local req_body = ngx.req.get_body_data()\n    if req_body == \"{\\\"a\\\":\\\"b\\\"}\"  then\n  return\n else\n  ngx.exit(ngx.HTTP_BAD_REQUEST)\n end\n end\n"]
+        }
+    },
+    "upstream": {
+        "nodes": {
+            "127.0.0.1:6666": 1
+        },
+        "type": "roundrobin"
+    }
+}'
+    '''
+    subprocess.Popen(command, stdout=subprocess.PIPE, stderr=subprocess.PIPE, shell=True)
+
+def main():
+    fw = open(cur_dir() + "/test.log",'wb')
+    fuzz_loggers = [FuzzLoggerText(file_handle=fw)]
+    session = Session(
+        target=Target(
+            connection=TCPSocketConnection("127.0.0.1", 9080, send_timeout=5.0, recv_timeout=5.0, server=False)
+        ),
+        fuzz_loggers=fuzz_loggers,
+        keep_web_open=False,
+    )
+
+    s_initialize(name="Request")
+    with s_block("Request-Line"):
+        s_group("Method", ["GET", "HEAD", "POST", "PUT", "DELETE", "CONNECT", "OPTIONS", "TRACE"])
+        s_delim(" ", name="space-1")
+        s_string("/post", name="Request-URI")
+        s_delim(" ", name="space-2")
+        s_string("HTTP/1.1", name="HTTP-Version")
+        s_static("\r\n", name="Request-Line-CRLF")
+        s_string("Host:", name="Host-Line")
+        s_delim(" ", name="space-3")
+        s_string("127.0.0.1:9080", name="Host-Line-Value")
+        s_static("\r\n", name="Host-Line-CRLF")
+        s_static('User-Agent', name='User-Agent-Header')
+        s_delim(':', name='User-Agent-Colon-1')
+        s_delim(' ', name='User-Agent-Space-1')
+        s_string('Mozilla/5.0 (Windows NT 6.2; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3223.8 Safari/537.36', name='User-Agent-Value')
+        s_static('\r\n', name='User-Agent-CRLF'),
+        s_static('Accept', name='Accept-Header')
+        s_delim(':', name='Accept-Colon-1')
+        s_delim(' ', name='Accept-Space-1')
+        s_string('text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8', name='Accept-Value')
+        s_static('\r\n', name='Accept-CRLF')
+        s_static("Content-Length:", name="Content-Length-Header")
+        s_delim(" ", name="space-4")
+        s_size("Body-Content", output_format="ascii", name="Content-Length-Value")
+        s_static("\r\n", "Content-Length-CRLF")
+        s_static('Connection', name='Connection-Header')
+        s_delim(':', name='Connection-Colon-1')
+        s_delim(' ', name='Connection-Space-1')
+        s_group('Connection-Type', ['keep-alive', 'close'])
+        s_static('\r\n', 'Connection-CRLF')
+        s_static('Content-Type', name='Content-Type-Header')
+        s_delim(':', name='Content-Type-Colon-1')
+        s_delim(' ', name='Content-Type-Space-1')
+        s_string('application/x-www-form-urlencoded', name='Content-Type-Value')
+        s_static('\r\n', name='Content-Type-CRLF')
+    s_static("\r\n", "Request-CRLF")
+
+    with s_block("Body-Content"):
+        s_string('{"a":"b"}', name="Body-Content-Value")
+
+    session.connect(s_get("Request"))
+    session.fuzz()
+
+if __name__ == "__main__":
+    # before test
+    create_route()
+    r1 = check_process()
+    main()
+    # after test
+    boofuzz_log = cur_dir() + "/test.log"
+    apisix_errorlog = "~/work/apisix/apisix/logs/error.log"
+    apisix_accesslog = "~/work/apisix/apisix/logs/access.log"
+    check_log(boofuzz_log, apisix_errorlog, apisix_accesslog)
+    r2 = check_process()
+    if r2 != r1:
+        print("before test, nginx's process list:%s,\nafter test, nginx's process list:%s"%(r1,r2))
+        raise AssertionError
diff --git a/t/fuzzing/route_test.py b/t/fuzzing/simpleroute_test.py
similarity index 100%
rename from t/fuzzing/route_test.py
rename to t/fuzzing/simpleroute_test.py

From 95f5afe08fd60b8038fd781f164e94441e63a7db Mon Sep 17 00:00:00 2001
From: idbeta <idbeta@gmail.com>
Date: Tue, 23 Feb 2021 13:37:38 +0800
Subject: [PATCH 2/4] remove trailing spaces

---
 .github/workflows/fuzzing-ci.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/fuzzing-ci.yaml b/.github/workflows/fuzzing-ci.yaml
index 634b8fb2e612..04b30d190f74 100644
--- a/.github/workflows/fuzzing-ci.yaml
+++ b/.github/workflows/fuzzing-ci.yaml
@@ -54,7 +54,7 @@ jobs:
     - name: run simpleroute test
       run: |
         python $PWD/t/fuzzing/simpleroute_test.py
-    
+
     - name: run serverless route test
       run: |
         python $PWD/t/fuzzing/serverless_route_test.py

From 6dfc821d6bcac48ecd592bbaa2206b2b8402ac25 Mon Sep 17 00:00:00 2001
From: idbeta <idbeta@gmail.com>
Date: Tue, 23 Feb 2021 13:50:33 +0800
Subject: [PATCH 3/4] remove useless file

---
 t/fuzzing/public.pyc | Bin 1743 -> 0 bytes
 1 file changed, 0 insertions(+), 0 deletions(-)
 delete mode 100644 t/fuzzing/public.pyc

diff --git a/t/fuzzing/public.pyc b/t/fuzzing/public.pyc
deleted file mode 100644
index 5d8b6a0677579d07301a2b74cadd1c1f77a791ce..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 1743
zcmbtUQEwYX5T3oW-PlcN1t<_Iq$5fd3n_7g#9LK@XjSnNMQ0EwQe`f88+$Kj-}QFg
zrp10CNIb*;<5%zl;QRLM<ON=c&pY?+?9ATm%r_hUwbj#qMn9#p`UQ9&W7uyXGW?Yk
zB4;;P$(dT~133#M3*=WAgp!3Gv?ObJ(5COO-^M=XZ~RVfz7ayj?QI0%L~<p0um+*H
zmRu|T;lDHNGolj%A$~8%d1+xJQYCKU_yJoL#<>lkS|^29b21uTm_m<6ePFp2_7Ky-
z_g2?-V6t=VlEL)t*tkJb8EdWvZg4Sw_s-15gKB<WnDnrE>+ZoNo!6tx)E@y_Mj^@{
ziAx+WFzjy-3!k=~rSd3{on_!r2!uSfWT9j`Ys(_Qt0Rk0w#T6?TXOY67A<-9@+(=k
zSNT{VjrRnH&NjRzL}G%xC3#z}MV9dHNY<6hK<u%&j{KtJ)v^45$_@KH#L{HWC_uxQ
zEu&8p7d^JoA{y7aiVl)!_aZUHuKoC5*K}Q%^?xl!UmhJryG;=$x+o9=KH{=+?oe4|
zotbfr#LmyFx=gjT-ukqx^vqZByLzrYIQ{nYdF<_3TvRf1?*V*mQ7LE2*>eWcY)FHm
z99wrE;JhqZun`pQ0p?8^M?P-q9^UGblmSr59nxuL9oY!rn4<b_B<XUiiB2yW$|smu
z#xJVJCwQc`)P6%5a+U=3UGO^qsJ>kH0_oGV1FDX8K!X6Z1d(2=kgP(#phvJVGMRS-
z<fLjt@-DS`kFseYGB1$)zJPA@7exkk*r1@@GZ=w(;9S1x?Y8KqFdzCfpV%J}TV<mI
zeX(ZCx;5sNbtutvFdLiM)nQuBZcL_$b-G?V9K7~4TSUp5%V_`Ssx~th?R~M_UsDBj
z@>P#X9mgEj8m(r{Yn`$vq-}^X#IQRM6W?MlAIK^fB7X$(s)L}hD1G|)s#YfRw}-zn
z4v&0=t2<TLGP);LyR@8EPd~@RUj$w0v(PbDV{$#-q#!pv&)y`sop`#+&YdazWpcXe
z1)dBUX)>@y4}YP#+Eo5RODCqtYP1RslPrsw0e=(59|FaX=;HTIXcd3PIh$zJY<D`Z
zV?Yg~SKIADVcAfkI$(28?Wy<GLp*)P&}4wC@uw6#O1w?+7~(b*kN(d!yn$_=Ov`Lu
S=%*h5TV{Jx;R(87xAzYP%WDe&


From 697d970bde12dbde2e15cbcd51ccf66d78ffc15c Mon Sep 17 00:00:00 2001
From: idbeta <idbeta@gmail.com>
Date: Tue, 23 Feb 2021 17:08:23 +0800
Subject: [PATCH 4/4] fixed bug

---
 t/fuzzing/public.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/t/fuzzing/public.py b/t/fuzzing/public.py
index e91a49aa096b..00f4ed20c33b 100644
--- a/t/fuzzing/public.py
+++ b/t/fuzzing/public.py
@@ -29,7 +29,7 @@ def check_log(*logs):
     apisix_errorlog = logs[1]
     apisix_accesslog = logs[2]
 
-    cmds = ['cat %s | grep -a "fail"'%boofuzz_log, 'cat %s | grep -a "error"'%apisix_errorlog, 'cat %s | grep -a " 500 "'%apisix_accesslog]
+    cmds = ['cat %s | grep -a "fail"'%boofuzz_log, 'cat %s | grep -a "error" | grep -v "invalid request body"'%apisix_errorlog, 'cat %s | grep -a " 500 "'%apisix_accesslog]
     for cmd in cmds:
         r = subprocess.Popen(cmd, stdout=subprocess.PIPE, shell=True)
         assert r.stdout.read().strip() == ""