Haven't receive it,yet. Re-send it in case of anything wrong happened.
> Hi, Jim,
>
> The webrev is updated, and please see in line.
>
> http://cr.opensolaris.org/~jxzhao/snort/webrev
>> Jason Zhao wrote:
>>>
>>> The new webrev is updated, please review it again and tell me your
>>> comments.
>>> http://cr.opensolaris.org/~jxzhao/snort/webrev
>>
>> usr/src/cmd/snort/Patches/snort.conf.patch
>> - I would keep all these lines and just
>> make them all comments so it is easy for
>> the user to include desired rules files.
>>
>> 40 -include $RULE_PATH/local.rules
>> 41 -include $RULE_PATH/bad-traffic.rules
>> 42 -include $RULE_PATH/exploit.rules
>> 43 -include $RULE_PATH/scan.rules
>> 44 -include $RULE_PATH/finger.rules
>> 45 -include $RULE_PATH/ftp.rules
>> 46 -include $RULE_PATH/telnet.rules
>> 47 -include $RULE_PATH/rpc.rules
>> 48 -include $RULE_PATH/rservices.rules
>> 49 -include $RULE_PATH/dos.rules
>> 50 -include $RULE_PATH/ddos.rules
>> 51 -include $RULE_PATH/dns.rules
>> 52 -include $RULE_PATH/tftp.rules
>> 53 -
>> 54 -include $RULE_PATH/web-cgi.rules
>> 55 -include $RULE_PATH/web-coldfusion.rules
>> 56 -include $RULE_PATH/web-iis.rules
>> 57 -include $RULE_PATH/web-frontpage.rules
>> 58 -include $RULE_PATH/web-misc.rules
>> 59 -include $RULE_PATH/web-client.rules
>> 60 -include $RULE_PATH/web-php.rules
>> 61 -
>> 62 -include $RULE_PATH/sql.rules
>> 63 -include $RULE_PATH/x11.rules
>> 64 -include $RULE_PATH/icmp.rules
>> 65 -include $RULE_PATH/netbios.rules
>> 66 -include $RULE_PATH/misc.rules
>> 67 -include $RULE_PATH/attack-responses.rules
>> 68 -include $RULE_PATH/oracle.rules
>> 69 -include $RULE_PATH/mysql.rules
>> 70 -include $RULE_PATH/snmp.rules
>> 71 -
>> 72 -include $RULE_PATH/smtp.rules
>> 73 -include $RULE_PATH/imap.rules
>> 74 -include $RULE_PATH/pop2.rules
>> 75 -include $RULE_PATH/pop3.rules
>> 76 -
>> 77 -include $RULE_PATH/nntp.rules
>> 78 -include $RULE_PATH/other-ids.rules
>> 79 -# include $RULE_PATH/web-attacks.rules
>> 80 -# include $RULE_PATH/backdoor.rules
>> 81 -# include $RULE_PATH/shellcode.rules
>> 82 -# include $RULE_PATH/policy.rules
>> 83 -# include $RULE_PATH/porn.rules
>> 84 -# include $RULE_PATH/info.rules
>> 85 -# include $RULE_PATH/icmp-info.rules
>> 86 -# include $RULE_PATH/virus.rules
>> 87 -# include $RULE_PATH/chat.rules
>> 88 -# include $RULE_PATH/multimedia.rules
>> 89 -# include $RULE_PATH/p2p.rules
>> 90 -# include $RULE_PATH/spyware-put.rules
>> 91 -# include $RULE_PATH/specific-threats.rules
>> 92 -include $RULE_PATH/experimental.rules
>> 93 -
>> 94 -# include $PREPROC_RULE_PATH/preprocessor.rules
>> 95 -# include $PREPROC_RULE_PATH/decoder.rules
>> 96 -
>> 97 # Include any thresholding or suppression commands. See
>> threshold.conf in the
>> 98 # <snort src>/etc directory for details. Commands don't
>> necessarily need to be
>> 99 # contained in this conf, but a separate conf makes it easier
>> to maintain them.
>>
> Done! Thank you.
>>
>> usr/src/pkgdefs/SUNWsnortu/depend
>> -shouldn't SUNWopensslr be included to support
>> when snort is run with ssl libraries?
> It seems like it doesn't depend on SUNWopensslr, from ldd output.
> ##########################################
> # ldd ./libsf_ssl_preproc.so
> libpcre.so.0 => /usr/lib/libpcre.so.0
> libpcap.so => /usr/lib/libpcap.so
> libm.so.2 => /usr/lib/libm.so.2
> libsocket.so.1 => /usr/lib/libsocket.so.1
> libnsl.so.1 => /usr/lib/libnsl.so.1
> libdl.so.1 => /usr/lib/libdl.so.1
> libc.so.1 => /usr/lib/libc.so.1
> libgcc_s.so.1 => /usr/sfw/lib/libgcc_s.so.1
> libdlpi.so.1 => /usr/lib/libdlpi.so.1
> libmp.so.2 => /usr/lib/libmp.so.2
> libmd.so.1 => /usr/lib/libmd.so.1
> libscf.so.1 => /usr/lib/libscf.so.1
> libinetutil.so.1 => /usr/lib/libinetutil.so.1
> libdladm.so.1 => /lib/libdladm.so.1
> libuutil.so.1 => /usr/lib/libuutil.so.1
> libgen.so.1 => /usr/lib/libgen.so.1
> libdevinfo.so.1 => /usr/lib/libdevinfo.so.1
> librcm.so.1 => /usr/lib/librcm.so.1
> libnvpair.so.1 => /usr/lib/libnvpair.so.1
> libexacct.so.1 => /usr/lib/libexacct.so.1
> libkstat.so.1 => /usr/lib/libkstat.so.1
> libcurses.so.1 => /usr/lib/libcurses.so.1
> libsec.so.1 => /usr/lib/libsec.so.1
> libavl.so.1 => /usr/lib/libavl.so.1
> libidmap.so.1 => /usr/lib/libidmap.so.1
> libldap.so.5 => /usr/lib/libldap.so.5
> libsldap.so.1 => /usr/lib/libsldap.so.1
> libadutils.so.1 => /usr/lib/libadutils.so.1
> libsasl.so.1 => /usr/lib/libsasl.so.1
> libnspr4.so => /usr/lib/mps/libnspr4.so
> libplc4.so => /usr/lib/mps/libplc4.so
> libnss3.so => /usr/lib/mps/libnss3.so
> libssl3.so => /usr/lib/mps/libssl3.so
> libresolv.so.2 => /usr/lib/libresolv.so.2
> libpthread.so.1 => /usr/lib/libpthread.so.1
> librt.so.1 => /usr/lib/librt.so.1
> libsoftokn3.so => /usr/lib/mps/libsoftokn3.so
> libplds4.so => /usr/lib/mps/libplds4.so
> libthread.so.1 => /usr/lib/libthread.so.1
> libbsm.so.1 => /usr/lib/libbsm.so.1
> libsecdb.so.1 => /usr/lib/libsecdb.so.1
> libtsol.so.2 => /usr/lib/libtsol.so.2
> ##########################################
>
> It invokes functions of libssl3.so which belongs to SUNWtls package.
> I have checked SUNWtls package, it doesn't depend on SUNWopensslr.
>
> The "make check_deps" shows no error.
> # make check_deps
> ......
> ## Validating control scripts.
> ## Packaging complete.
> /usr/perl5/bin/perl
> /builds1/xz201216/sfwnv_split/usr/src/tools/check-deps.pl -e
> /builds1/xz201216/sfwnv_split/usr/src/tools/exception_list.check-deps
> -d /builds1/xz201216/sfwnv_split/packages/i386/nightly-nd -p
> /net/netinstall/export/nv/x/latest/Solaris_11/Product SUNWsnortu
> #
>
> The test shows (ssl plugin) it could work without /lib/libssl.so(which
> is a library of SUNWopensslr).
> ###########################
> ......
> SSL Preprocessor:
> SSL packets decoded: 4 Client Hello: 1
> Server Hello: 1 Certificate: 0
> Server Done: 2 Client Key Exchange: 0
> Server Key Exchange: 0 Change Cipher: 2
> Finished: 0 Client Application: 1
> Server Application: 1 Alert: 0
> Unrecognized records: 0 Completed handshakes: 0
> Bad handshakes: 0 Sessions ignored: 0 Detection
> disabled: 0 ......
> ###########################
>
>
> Thanks
> Jason
>
