On 6/10/07, Ian Brown <ianbrn@xxxxxxxxx> wrote:
what beats me is that the previous version even compiled.... was there any other daddr variable?
Hello,
Thnks, Mulyadi
>> May we know, what does "daddr" refer to?
oops ! I my bad !
Instead : if (htonl(0xC0A80067) == daddr)
it should be:
if (htonl(0xC0A80067) == iph->daddr)
what beats me is that the previous version even compiled.... was there any other daddr variable?
daddr is a field of ip header (struct iphdr) defined thus:
__be32 daddr
See: struct iphdr definition in include/linux/ip.h.
Regards,
Ian
On 6/9/07, Mulyadi Santosa < mulyadi.santosa@xxxxxxxxx> wrote:
> Hi..
> >
> > Now, how should I comapare the value of daddr (which is __be32) in my
> > module
> > to that specified ip : " 192.168.0.103"?
> >
> > I tried the following:
> > ...
> > struct iphdr *iph;
> > iph = skb->nh.iph;
> > if (iph)
> > {
> > if (htonl(0xC0A80067) == daddr)
> > printk("got daddr=192.168.0.103\n");
> > else
> > printk("did not get daddr=192.168.0.103\n ");
> > }
> >
> > And I get **wrong** answer , namely it prints: "did not get daddr" , even
> > though I know for sure that the daddr of the ip header is 192.168.0.103.
> Interesting. May we know, what does "daddr" refer to? I mean, clearly
> that seems like local variable or local variable and the real "daddr" is
> in the skb. So, are you sure it had pointed to the correct "daddr" ?
> >
> > I think that the hex representation for
> > 192.168.0.103 is 0xC0A80067; am I right in this ? (I caclulated this
> > thus:
> > C0=192, A8=168, 00=0, 67=103).
> > But maybe the order of the bytes should be different ? something like:
> > if (htonl(0x6700A8C0) == daddr)
> > printk("got daddr=192.168.0.103\n");
> > Or is there another way I should do this ?
> >
> AFAIK, you're correct. However, it won't hurt if you try some ordering
> possibility like you said above. Please notice that I absolutely know
> nothing about x86_64 byte ordering.
>
> Good luck.
>
> regards,
>
> Mulyadi
>
>
--
To unsubscribe from this list: send an email with
"unsubscribe kernelnewbies" to ecartis@xxxxxxxxxxxx
Please read the FAQ at http://kernelnewbies.org/FAQ
--
The box said "Requires Windows Vista or better." So I installed LINUX
