diff --git a/modules/auth/repo_form.go b/modules/auth/repo_form.go
index a6c71e99e..1b1a4f712 100644
--- a/modules/auth/repo_form.go
+++ b/modules/auth/repo_form.go
@@ -71,7 +71,7 @@ func (f *RepoSettingForm) Validate(ctx *macaron.Context, errs *binding.Errors, l
 type NewWebhookForm struct {
 	Url         string `form:"url" binding:"Required;Url"`
 	ContentType string `form:"content_type" binding:"Required"`
-	Secret      string `form:"secret""`
+	Secret      string `form:"secret"`
 	PushOnly    bool   `form:"push_only"`
 	Active      bool   `form:"active"`
 }
diff --git a/modules/log/file.go b/modules/log/file.go
index cce535296..52a2f68c7 100644
--- a/modules/log/file.go
+++ b/modules/log/file.go
@@ -34,7 +34,7 @@ type FileLogWriter struct {
 
 	// Rotate daily
 	Daily          bool  `json:"daily"`
-	Maxdays        int64 `json:"maxdays`
+	Maxdays        int64 `json:"maxdays"`
 	daily_opendate int
 
 	Rotate bool `json:"rotate"`
diff --git a/modules/social/social.go b/modules/social/social.go
index 326a463fa..f4d859882 100644
--- a/modules/social/social.go
+++ b/modules/social/social.go
@@ -391,5 +391,4 @@ func (s *SocialWeibo) UserInfo(token *oauth.Token, _ *url.URL) (*BasicUserInfo,
 		Identity: token.Extra["id_token"],
 		Name:     data.Name,
 	}, nil
-	return nil, nil
 }