Re-used processes don't require login

Bug #1227273 reported by Thomas Berezansky
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
SIPServer
Fix Released
High
Unassigned

Bug Description

When a new connection gets a process that had previously logged in the client can skip the login process entirely and continue on as though they were the previous user on that process.

I have an apparently working fix in the new SIPServer security repo (<email address hidden>:security/SIPServer) in a branch named "reset_account_info".

Tags: pullrequest
Revision history for this message
Galen Charlton (gmc) wrote :

I've reproduced the problem (by running a SIP server that has max_servers set to 1) and successfully tested Thomas's patch.

I've pushed a signed-off version of it to security/reset_account_info_signedoff along with a follow-up that does two things:

- makes explicit the already implicit requirement that 93 be the first message presented when using the raw transport
- tweak the logging for clients that break that convention -- hopefully this will make it easier to identify clients that don't attempt to log in first.

tags: added: pullrequest
Revision history for this message
Galen Charlton (gmc) wrote :

From my POV, while it would be nice if my follow-up got pushed (assuming successful testing), Thomas' patch by itself is sufficient to close the security hole.

Revision history for this message
Bill Erickson (berick) wrote :

Sign-off to Galen's patch pushed to security/reset_account_info_signedoff2

Revision history for this message
Galen Charlton (gmc) wrote :

Pushed Thomas' patch to master. My follow-up patch is now superseded in part by the 99-then-33 work.

information type: Private Security → Public
Changed in sipserver:
status: New → Fix Committed
Galen Charlton (gmc)
Changed in sipserver:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.