Added more bounds checking to revised get_TLS_SNI and replaced remaining casts on two bytes with two_bytes_to_short call.

May be this will also fix renaming issues with BSD port?? See #810

Added more bounds checking to revised get_TLS_SNI and replaced remaining casts on two bytes with two_bytes_to_short call.

May be this will also fix renaming issues with BSD port?? See #810

i can test with bsd and i will

Added more bounds checking to revised get_TLS_SNI and replaced remaining casts on two bytes with two_bytes_to_short call.
May be this will also fix renaming issues with BSD port?? See #810

i can test with bsd and i will

it looking good, i did delete unsignedchar flag and i did test , sometimes i get err connection closed and i never seen broke url in access.log

I'm not sure if the is the same issue as this and #810, but the log truncates the url and puts garbage characters. For example:

"2024.10.23 19:36:49","","192.168.1.118","https://update.goog�����","*NETERROR* Failed to negotiate ssl connection to client","","0","0","SSL SITE","1","403","","192.168.1.118","group1","","","-","-",""

I'm running Linux Mint 20.2 and e2guardian 5.5.5, installed from deb package, with transparent proxy. This occurs a few times per day.