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 /api4/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 'api4/team.go')
| -rw-r--r-- | api4/team.go | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/api4/team.go b/api4/team.go index 58dcaca0d..fe5d7a266 100644 --- a/api4/team.go +++ b/api4/team.go @@ -623,10 +623,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 } @@ -651,11 +655,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 { |