diff options
author | Chris <ccbrown112@gmail.com> | 2017-11-03 10:25:38 -0500 |
---|---|---|
committer | Corey Hulen <corey@hulen.com> | 2017-11-03 08:25:38 -0700 |
commit | d5dbdb273703a3add0f1c3287aa531854ab30269 (patch) | |
tree | 6b1fb9079f9fae9d25bdb7ea459261383ee720d6 /api/team.go | |
parent | 9d32cd42085bbb37460d815c6c1a00ad881c4895 (diff) | |
download | chat-d5dbdb273703a3add0f1c3287aa531854ab30269.tar.gz chat-d5dbdb273703a3add0f1c3287aa531854ab30269.tar.bz2 chat-d5dbdb273703a3add0f1c3287aa531854ab30269.zip |
several one-line panic, race, and logic fixes (#7766)
Diffstat (limited to 'api/team.go')
-rw-r--r-- | api/team.go | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/api/team.go b/api/team.go index 7f53786d9..089680624 100644 --- a/api/team.go +++ b/api/team.go @@ -386,10 +386,14 @@ func importTeam(c *Context, w http.ResponseWriter, r *http.Request) { } importFromArray, ok := r.MultipartForm.Value["importFrom"] + if !ok || len(importFromArray) < 1 { + c.Err = model.NewAppError("importTeam", "api.team.import_team.no_import_from.app_error", nil, "", http.StatusBadRequest) + return + } importFrom := importFromArray[0] fileSizeStr, ok := r.MultipartForm.Value["filesize"] - if !ok { + if !ok || len(fileSizeStr) < 1 { c.Err = model.NewAppError("importTeam", "api.team.import_team.unavailable.app_error", nil, "", http.StatusBadRequest) return } @@ -414,11 +418,11 @@ func importTeam(c *Context, w http.ResponseWriter, r *http.Request) { fileInfo := fileInfoArray[0] fileData, err := fileInfo.Open() - defer fileData.Close() if err != nil { c.Err = model.NewAppError("importTeam", "api.team.import_team.open.app_error", nil, err.Error(), http.StatusBadRequest) return } + defer fileData.Close() var log *bytes.Buffer switch importFrom { |