Merge pull request #726 in SNORT/snort3 from appid_ptypes_scan_patch to master

Merge pull request #726 in SNORT/snort3 from appid_ptypes_scan_patch to master

Squashed commit of the following:

commit cfbad0aea0e04b034f7bcd70d07de6fcfc36dc73
Author: davis mcpherson <davmcphe.cisco.com>
Date:   Mon Nov 28 10:50:23 2016 -0500

    delete auto ptr used to point to member variable of the AppIdSession class, just use the member variable directly.  Some coding style improvements

commit 616baeb1eae80e6d4954c0a1e85db4d34085a13e
Author: davis mcpherson <davmcphe.cisco.com>
Date:   Mon Nov 28 15:06:26 2016 -0500

    make ptype_scan_counts a field of the httpSession struct

commit fef9bdf71276aa9b8966609c49743f6df3136bcd
Author: davis mcpherson <davmcphe.cisco.com>
Date:   Mon Nov 28 14:41:55 2016 -0500

    Add mutex lock around calls into crypto lib for X509 cert processing, crypto lib not thread safe so calls into this lib from multiple packet threads appear to be causing random failures