Mark, Here you go...
Gilbert, This copy has the change that was requested. Check it in.
If you have any questions/comments let me know and I will update my copy.
Jeff Foster
jfoste@xxxxxxxxxxxx
-----Original Message-----
From: Clayton, Mark [mailto:mclayton@xxxxxxxxxxxx]
Sent: Thursday, November 02, 2000 9:16 AM
To: 'Jeff Foster'
Subject: RE: [Ethereal-dev] AIM / OSCAR dissector
Jeff, I looked through the maillist archive and my inbox,
but I can't find your post. Would you send me a copy?
I want to start a dissector.
Much thanks,
Mark
-----Original Message-----
From: Jeff Foster [mailto:jfoste@xxxxxxxxxxxx]
Sent: Thursday, November 02, 2000 10:08 AM
To: 'Ralf Holzer'; ethereal-dev@xxxxxxxxxxxx
Subject: RE: [Ethereal-dev] AIM / OSCAR dissector
per Ralf Holzer -
> I just wrote a dissector for the OSCAR protocol used by AOL Instant
> Messenger. (OSCAR is the closed protocol used by the "official" clients).
>
> The code is probably not quite ready for prime time, but it dissects the
most
> important parts of OSCAR traffic. There are still some issues with the
> correct offsets for the messages, because they vary from client to client.
>
> I know there is lots of room for improvement, but I would appreciate it,
if I
> could get some feedback on it.
>
Welcome to ethereal development.
One thing that I see is you are using the old style dissectors not the new
(and
improved) tvbuffer style dissectors. It looks like the code was cut and
paste
from the README.developer so it isn't really your fault. I posted a new
version
of README.developer to the mailing list last week. I thought it was going to
make
the 8.13 release, however it didn't and isn't in the CVS tree. Our mistake.
I can
send you a copy of it if you want. I know this will more changes then you
expect,
but we really need to write dissectors using tvbuffers.
Thanks for the code.
Jeff Foster
jfoste@xxxxxxxxxxxx
_______________________________________________
Ethereal-dev mailing list
Ethereal-dev@xxxxxxxxxxxx
http://www.ethereal.com/mailman/listinfo/ethereal-dev
Attachment:
README.developer
Description: Binary data
