> 13 окт. 2017 г., в 18:39, Stuart Henderson <s...@spacehopper.org> написал(а):
> 
> On 2017/10/13 13:49, Stuart Henderson wrote:
>> Thanks. So looking at this and the source code, it shows it is using strchr()
>> to look for the position of a space character in the payload data. But the 
>> str*
>> functions expect a NUL-terminated string, and the string here is not, so 
>> strchr
>> searches beyond the end of the payload, looking for either a NUL or a space.
>> 
>> This code hasn't changed in 3.0, so the problem will still exist there.
>> 
>> I have created an issue upstream: https://github.com/ntop/ntopng/issues/1518
>> 
> 
> Upstream have committed a fix to their repository, which I have
> applied to the ports tree.
> 
> If you don't have the ports tree on your system already, follow "Getting
> the ports and xenocara trees" on https://www.openbsd.org/anoncvs.html
> (I have committed it to the -current and 6.2-stable branches).
> 
> Before building, "pkg_add libtool autoconf%2.69 automake%1.15 gmake"
> to save time building dependencies, then "cd /usr/ports/net/ntopng &&
> make package && doas make update".
> 
> Alternatively if you use -current snapshots it will show up in the
> binary packages in a few days.
> 

Hello again! I just built port and start testing it again, 
This time it crashed with this:

Program received signal SIGSEGV, Segmentation fault.
[Switching to thread 118222]
0x000006de1f6a6397 in ndpi_search_bgp () from /usr/local/bin/ntopng
Current language:  auto; currently minimal

Here is bt full:

#0  0x000006de1f6a6397 in ndpi_search_bgp () from /usr/local/bin/ntopng
No symbol table info available.
#1  0x000006de1f6a210b in check_ndpi_tcp_flow_func ()
   from /usr/local/bin/ntopng
No symbol table info available.
#2  0x000006de1f6a2f68 in ndpi_detection_process_packet ()
   from /usr/local/bin/ntopng
No symbol table info available.
#3  0x000006de1f64c50e in NetworkInterface::processPacket (
    this=0x6e0d1b8a1d0, when=0x6e1011b07e8, time=1508133385021, 
    eth=0x6e0b51f0fbe, vlan_id=0, iph=0x6e0b51f0fcc, ip6=0x0, ipsize=46, 
    rawsize=60, h=0x6e1011b07e8, packet=0x6e0b51f0fbe "", 
    shaped=0x6e04d0b714d, ndpiProtocol=0x6e04d0b714e)
    at src/NetworkInterface.cpp:897
        ndpi_flow = (ndpi_flow_struct *) 0x6e0bf67d800
        cli = (struct ndpi_id_struct *) 0x6e09236df00
        srv = (struct ndpi_id_struct *) 0x6e107270c00
        src2dst_direction = true
        l4_proto = 6 '\006'
        flow = (class Flow *) 0x6e09f185c00
        eth_src = (u_int8_t *) 0x6e0b51f0fc4 "T\004??\033?\b"
        eth_dst = (u_int8_t *) 0x6e0b51f0fbe ""
        src_ip = {addr = {ipVersion = 4 '\004', localHost = 0 '\0', 
    privateIP = 1 '\001', multicastIP = 0 '\0', broadcastIP = 0 '\0', 
    notUsed = 0 '\0', ipType = {ipv6 = {u6_addr = {
          u6_addr8 = 0x6e04d0b6d0c "??\003?", u6_addr16 = 0x6e04d0b6d0c, 
          u6_addr32 = 0x6e04d0b6d0c}}, ipv4 = 3523455168}}, 
  ip_key = 3232236498}
        dst_ip = {addr = {ipVersion = 4 '\004', localHost = 0 '\0', 
    privateIP = 0 '\0', multicastIP = 0 '\0', broadcastIP = 0 '\0', 
    notUsed = 0 '\0', ipType = {ipv6 = {u6_addr = {
          u6_addr8 = 0x6e04d0b6cf4 "??p\v", u6_addr16 = 0x6e04d0b6cf4, 
          u6_addr32 = 0x6e04d0b6cf4}}, ipv4 = 191949268}}, 
  ip_key = 3572068363}
        src_port = 24282
        dst_port = 36608
        payload_len = 6
        tcph = (ndpi_tcphdr *) 0x6e0b51f0fe0
        udph = (ndpi_udphdr *) 0x0
        l4_packet_len = 26
        l4 = (u_int8_t *) 0x6e0b51f0fe0 "?^"
        tcp_flags = 24 '\030'
        payload = (u_int8_t *) 0x6e0b51f0ff4 "DONE\r\n"
        ip = (u_int8_t *) 0x6e0b51f0fcc "E"
        is_fragment = false
        new_flow = true
        pass_verdict = true
        a_shaper_id = 0
        b_shaper_id = 0
#4  0x000006de1f64e81d in NetworkInterface::dissectPacket (
    this=0x6e0d1b8a1d0, h=0x6e1011b07e8, packet=0x6e0b51f0fbe "", 
    shaped=0x6e04d0b714d, ndpiProtocol=0x6e04d0b714e)
    at src/NetworkInterface.cpp:1403
        frag_off = 16384
        iph = (ndpi_iphdr *) 0x6e0b51f0fcc
        ip6 = (ndpi_ipv6hdr *) 0x0
        ba = (class std::bad_alloc &) @0x6e049dfaa20: {<std::exception> = {
    _vptr$exception = 0x6e085c1eec0}, <No data fields>}
        srcHost = (class Host *) 0x6e1094ca000
        dstHost = (class Host *) 0x6e078b60000
        lasttime = 1508133385021
        oom_warning_sent = false
        oom_warning_sent = false
        ethernet = (ndpi_ethhdr *) 0x6e0b51f0fbe
        dummy_ethernet = {h_dest = 0x6e04d0b7050 "\200p\vM?\006", 
  h_source = 0x6e04d0b7056 "", h_proto = 1760}
---Type <return> to continue, or q <return> to quit---
        time = 1508133385021
        eth_type = 2048
        ip_offset = 14
        vlan_id = 0
        eth_offset = 0
        null_type = 2515945984
        pcap_datalink_type = 1
        pass_verdict = true
#5  0x000006de1f602e3b in _ZL14packetPollLoopPv (ptr=0x6e0d1b8a1d0)
    at src/PcapInterface.cpp:187
        p = 37
        shaped = false
        pkt = (const u_char *) 0x6e0b51f0fbe ""
        hdr = (pcap_pkthdr *) 0x6e1011b07e8
        rc = 1
        iface = (PcapInterface *) 0x6e0d1b8a1d0
        pd = (pcap_t *) 0x6e1011b0600
        pcap_list = (FILE *) 0x0
#6  0x000006e042a33cae in _rthread_start (v=Variable "v" is not available.
)
    at /usr/src/lib/librthread/rthread.c:96
        retval = (void *) 0x6e0b51f0ff4
#7  0x000006e07a1bce0b in __tfork_thread ()
    at /usr/src/lib/libc/arch/amd64/sys/tfork_thread.S:75
No locals.
#8  0x0000000000000000 in ?? ()
No symbol table info available.

Reply via email to