search

bstansell / conserver

Logged, multi-user access to device consoles
https://www.conserver.com/
BSD 3-Clause "New" or "Revised" License
129 stars 38 forks source link

Updated conserver/cutil.c and conserver/group.c to use snprintf( ) vi… #92

Open agialluc opened 11 months ago

agialluc commented 11 months ago

Updated conserver/cutil.c and conserver/group.c to use snprintf( ) vice sprintf( ) to prevent buffer overflows.

This code change should be carefully checked. I am not normally a coder.

agialluc commented 11 months ago

I should note, for the record, this may not be the root cause, but is just my suspicion. Regardless, I do think snprintf( ) calls are an improvement.

agialluc commented 11 months ago

I was a beautiful theory while it lasted. Please feel free to abandon this merge as necessary.