[sr-dev] git:master:de8c85ce: pv: pv_get_method fixed error "pv_get_method(): no CSEQ header"

[Daniel-Constantin Mierla]

Content preview: Module: kamailio Branch: master Commit: de8c85ceae93779cdf7d6aea2eabb7567d69d7ce
URL: https://github.com/kamailio/kamailio/commit/de8c85ceae93779cdf7d6aea2eabb7567d69d7ce
Author: Sergey Safarov <***@gmail.com> Committer: Daniel-Constantin
Mierla <***@gmail.com> Date: 2021-06-04T16:22:49+02:00

Content analysis details: (-2.4 points, 5.5 required)

pts rule name description
---- ---------------------- --------------------------------------------------
-1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP
-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%
[score: 0.0000]
0.0 DKIM_ADSP_CUSTOM_MED No valid author signature, adsp_override is
CUSTOM_MED
0.2 HEADER_FROM_DIFFERENT_DOMAINS From and EnvelopeFrom 2nd level
mail domains are different
0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail
provider (miconda[at]gmail.com)
1.0 FORGED_GMAIL_RCVD 'From' gmail.com does not match 'Received'
headers
0.2 FREEMAIL_FORGED_FROMDOMAIN 2nd level domains in From and
EnvelopeFrom freemail headers are
different
-1.0 MAILING_LIST_MULTI Multiple indicators imply a widely-seen list
manager

Module: kamailio
Branch: master
Commit: de8c85ceae93779cdf7d6aea2eabb7567d69d7ce
URL: https://github.com/kamailio/kamailio/commit/de8c85ceae93779cdf7d6aea2eabb7567d69d7ce

Author: Sergey Safarov <***@gmail.com>
Committer: Daniel-Constantin Mierla <***@gmail.com>
Date: 2021-06-04T16:22:49+02:00

pv: pv_get_method fixed error "pv_get_method(): no CSEQ header"

GH #2761: Error obeserver when sent HTTP reply like

HTTP/1.1 200 OK
Sia: SIP/2.0/TCP 8.8.8.8:39813
Content-Type: application/json
Server: kamailio
Content-Length: 49

{"data":{"status-code":200,"reason-phrase":"OK"}}

---

Modified: src/modules/pv/pv_core.c

---

Diff: https://github.com/kamailio/kamailio/commit/de8c85ceae93779cdf7d6aea2eabb7567d69d7ce.diff
Patch: https://github.com/kamailio/kamailio/commit/de8c85ceae93779cdf7d6aea2eabb7567d69d7ce.patch

---

diff --git a/src/modules/pv/pv_core.c b/src/modules/pv/pv_core.c
index 5b011ea248..1b39ef3fb8 100644
--- a/src/modules/pv/pv_core.c
+++ b/src/modules/pv/pv_core.c
@@ -147,6 +147,9 @@ int pv_get_method(struct sip_msg *msg, pv_param_t *param,
(int)msg->first_line.u.request.method_value);
}

+ if (IS_HTTP_REPLY(msg))
+ return pv_get_null(msg, param, res);
+
if(msg->cseq==NULL && ((parse_headers(msg, HDR_CSEQ_F, 0)==-1) ||
(msg->cseq==NULL)))
{