Browse Source

Fixed several possible nil pointer access issues.

pull/28/head
Simon Eisenmann 12 years ago
parent
commit
dfd319d06c
  1. 11
      src/app/spreed-speakfreely-server/sessions.go
  2. 2
      src/app/spreed-speakfreely-server/users.go

11
src/app/spreed-speakfreely-server/sessions.go

@ -83,11 +83,14 @@ func (sessions *Sessions) Patch(request *http.Request) (int, interface{}, http.H
error = true error = true
} }
var userid string
// Validate with users handler. // Validate with users handler.
userid, err := sessions.users.handler.Validate(&snr, request) if sessions.users.handler != nil {
if err != nil { userid, err = sessions.users.handler.Validate(&snr, request)
error = true if err != nil {
log.Println("Session patch failed - users validation failed.", err) error = true
log.Println("Session patch failed - users validation failed.", err)
}
} }
// Make sure that we have a user. // Make sure that we have a user.

2
src/app/spreed-speakfreely-server/users.go

@ -263,7 +263,7 @@ func NewUsers(hub *Hub, mode, realm string, runtime phoenix.Runtime) *Users {
var err error var err error
// Create handler based on mode. // Create handler based on mode.
if handler, err = users.createHandler(mode, runtime); err == nil { if handler, err = users.createHandler(mode, runtime); handler != nil && err == nil {
users.handler = handler users.handler = handler
// Register handler Get at the hub. // Register handler Get at the hub.
users.hub.useridRetriever = func(request *http.Request) (userid string, err error) { users.hub.useridRetriever = func(request *http.Request) (userid string, err error) {

Loading…
Cancel
Save