diff --git a/routers/user/user.go b/routers/user/user.go
index 7cd8bca55..cf1314086 100644
--- a/routers/user/user.go
+++ b/routers/user/user.go
@@ -32,6 +32,14 @@ func SignIn(req *http.Request, r render.Render) {
 	}
 
 	// todo sign in
+	_, err := models.LoginUserPlain(req.FormValue("account"), req.FormValue("passwd"))
+	if err != nil {
+		r.HTML(200, "base/error", map[string]interface{}{
+			"Error": fmt.Sprintf("%v", err),
+		})
+		return
+	}
+	r.Redirect("/")
 }
 
 func SignUp(req *http.Request, r render.Render) {
@@ -61,9 +69,16 @@ func SignUp(req *http.Request, r render.Render) {
 	}
 
 	err = models.RegisterUser(u)
-	r.HTML(403, "status/403", map[string]interface{}{
-		"Title": fmt.Sprintf("%v", err),
-	})
+	if err != nil {
+		if err != nil {
+			r.HTML(200, "base/error", map[string]interface{}{
+				"Error": fmt.Sprintf("%v", err),
+			})
+			return
+		}
+	}
+
+	r.Redirect("/")
 }
 
 func Delete(req *http.Request, r render.Render) {
diff --git a/templates/base/error.tmpl b/templates/base/error.tmpl
new file mode 100644
index 000000000..8d3c36be9
--- /dev/null
+++ b/templates/base/error.tmpl
@@ -0,0 +1,6 @@
+{{template "base/head" .}}
+{{template "base/navbar" .}}
+<div id="gogs-body" class="container">
+	An error is occupied : {{.Error}}
+</div>
+{{template "base/footer" .}}
\ No newline at end of file
diff --git a/templates/user/signin.tmpl b/templates/user/signin.tmpl
index dcdeaa77c..c3a3131f5 100644
--- a/templates/user/signin.tmpl
+++ b/templates/user/signin.tmpl
@@ -1,7 +1,7 @@
 {{template "base/head" .}}
 {{template "base/navbar" .}}
 <div class="container" id="gogs-body">
-    <form action="/user/signup" method="post" class="form-horizontal" id="gogs-login-card">
+    <form action="/user/signin" method="post" class="form-horizontal" id="gogs-login-card">
         <h3>Log in</h3>
         <div class="form-group">
             <label class="col-md-4 control-label">Username or Email: </label>