==922== Thread 2:
==922== Conditional jump or move depends on uninitialised value(s)
==922== at 0x402AA52A: processIpPkt (pbuf.c:975)
==922== by 0x402AD189: processPacket (pbuf.c:2513)
==922== by 0x402AB37C: dequeuePacket (pbuf.c:1735)
==922== by 0x4045B5C2: thread_wrapper (vg_libpthread.c:667)
==922==
==922== Thread 2:
==922== Use of uninitialised value of size 4
==922== at 0x402AA536: processIpPkt (pbuf.c:975)
==922== by 0x402AD189: processPacket (pbuf.c:2513)
==922== by 0x402AB37C: dequeuePacket (pbuf.c:1735)
==922== by 0x4045B5C2: thread_wrapper (vg_libpthread.c:667)
if(srcHost->fingerprint == NULL) {
char fingerprint[64];
int WIN=0, MSS=-1, WS=-1, S=0, N=0, D=0, T=0;
int ttl;
char WSS[3], _MSS[5];
if (tp.th_flags & TH_SYN) /* only SYN or SYN-2ACK packets */
{
...
while(opt_ptr < tcp_data)
{
>>> switch(*opt_ptr)
{
...
}
}
}
...
}
Luca, I think valgrind is right, because the set of tp is this:
memcpy(&tp, bp+hlen, sizeof(struct tcphdr));
(pbuf.c @ 921), which doesn't copy any of the option fields. We should
probably add 40 bytes (that's the maximum length of the options field) to
that memcpy...
-----Burton
_______________________________________________
Ntop-dev mailing list
[EMAIL PROTECTED]
http://listgateway.unipi.it/mailman/listinfo/ntop-dev
