Wireshark · Ethereal-dev: Re: [Ethereal-dev] Re: developing of packet-kink.c

[Gilbert Ramirez <gilbertr@xxxxxxxxx>]

I can't build now that this is committed.

I don't have the MIT or Heimdal kerberos libs installed. So,
decrypt_krb5_data in packet-kerberos. does not get defined. The
reference to that function in packet-kink.c is then undefined.

The attached patch seems to work for me, but I'd like a review from
Takeshi to make sure it's ok.

--gilbert


> On 5/11/05, T.Nakashima@xxxxxxxxxxxxxxx <T.Nakashima@xxxxxxxxxxxxxxx> wrote:
> >
> > hi,all
> >
> > I'm developer of packet-kink.c.
> > I modified packet-kink.c etc and added following function
> > concern kink protocol this time.
> >
> > -parsing kink_isakmp as isakmp
> > -decrypting kink_encrypt by a keytab file.
> >
> > Could you add this function to ethereal?
> >
> > When I added this function, I modified the following sorce file.
> > -packet-kink.c
> > -packet-kerberos.c
> > -packet-kerberos.h
> > -packet-isakmp.c
> >
> > I send difference those sorce file and those of ethereal-0.10.11.
> >
> > best regards,
> >
> > ------------------
> > yokogawa electoric corp
> > Takeshi Nakashima
> >

Index: epan/dissectors/packet-kink.c
===================================================================
--- epan/dissectors/packet-kink.c	(revision 14456)
+++ epan/dissectors/packet-kink.c	(working copy)
@@ -736,6 +736,7 @@
   /* decrypt kink encrypt */
 
   if(keytype != 0){
+#ifdef HAVE_KERBEROS
     plaintext=decrypt_krb5_data(tree, pinfo, 0, encrypt_length, data_value, keytype);    
     if(plaintext){
       next_tvb=tvb_new_real_data(plaintext, encrypt_length, encrypt_length);
@@ -743,6 +744,7 @@
       add_new_data_source(pinfo, next_tvb, "decrypted kink encrypt");
       dissect_decrypt_kink_encrypt(pinfo, next_tvb, tree, encrypt_length);
     }
+#endif
   }
   else{
     inner_next_pload = tvb_get_guint8(tvb, offset);