<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:#0563C1;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:#954F72;
        text-decoration:underline;}
p.msonormal0, li.msonormal0, div.msonormal0
        {mso-style-name:msonormal;
        mso-margin-top-alt:auto;
        margin-right:0in;
        mso-margin-bottom-alt:auto;
        margin-left:0in;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
span.EmailStyle18
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.EmailStyle19
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.EmailStyle20
        {mso-style-type:personal-compose;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style>
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72">
<div class="WordSection1">
<p class="MsoNormal"><span lang="EN-IN">Hi,<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">There is a issue in PAWS window checks in snort_steam_tcp.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">Consider you got big timestamp in last segment and  current segment has 0(zero) or  less timestamp.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">Ex.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">ts_last = 2331162992 and tdb->ts = 0 (bad segment) and dry run below code.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">            if (validate_timestamp)<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">            {<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                int result = 0;<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                if (listener->tcp_policy->policy == STREAM_POLICY_LINUX)<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                {<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                    /* Linux 2.6 accepts timestamp values that are off<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                     * by one. */<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                    result = (int)((tdb->ts - talker->ts_last) + 1);<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                }<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                else<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                {<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                    result = (int)(tdb->ts - talker->ts_last);<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                }<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">Here we expect result should negative as tdb->ts is less than ts_last. But as last segment got big timestamp which actually negative number in 32bit integer and if we do 0 – 2331162992 will become positive number 1963804304,
 but expect  < 0.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                if(result < 0)<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                {<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                    STREAM_DEBUG_WRAP(DebugMessage(DEBUG_STREAM_STATE,<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                                "Packet outside PAWS window, dropping\n"););<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                    /* bail, we've got a packet outside the PAWS window! */<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                    //Discard();<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                    *eventcode |= EVENT_BAD_TIMESTAMP;<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                    if(listener->tcp_policy->flags & STREAM_CONFIG_ENABLE_ALERTS)<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                    NormalDropPacket(p);<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                    return ACTION_BAD_PKT;<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">                }<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">It must fall in this if and drop this bad segment, however it is bypassing this checks due to wrongly handled comparison.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">There are other checks after this, which actually updates the timestamp wrongly.
<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">As I see this should be change or am I missing something here. Happy to give patch if you agree.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">Thanks,<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-IN">Nilesh<o:p></o:p></span></p>
</div>
</body>
</html>