diff --git a/cmd/dm-master/main.go b/cmd/dm-master/main.go
index 7489e66ee26..d2f64c3c044 100644
--- a/cmd/dm-master/main.go
+++ b/cmd/dm-master/main.go
@@ -34,7 +34,7 @@ import (
 )
 
 func main() {
-	// 1. parse config
+	// 1. parse config.
 	cfg := master.NewConfig()
 	err := cfg.Parse(os.Args[1:])
 	switch errors.Cause(err) {
diff --git a/dm/tests/openapi/run.sh b/dm/tests/openapi/run.sh
index f231a000fbb..5bce9a196eb 100644
--- a/dm/tests/openapi/run.sh
+++ b/dm/tests/openapi/run.sh
@@ -1136,7 +1136,7 @@ function test_tls() {
 	# miss pd all certificate
 	openapi_task_check "create_noshard_task_with_security_failed" $task_name \
 		"$(cat $cur/tls_conf/ca2.pem)" "$(cat $cur/tls_conf/tidb.pem)" "$(cat $cur/tls_conf/tidb.key)" \
-		"" "" ""
+		"" "" "123"
 
 	killall -9 tidb-server 2>/dev/null || true
 	killall -9 tikv-server 2>/dev/null || true
diff --git a/dm/unit/unit.go b/dm/unit/unit.go
index 73613bbec5a..61c0db96199 100644
--- a/dm/unit/unit.go
+++ b/dm/unit/unit.go
@@ -40,6 +40,7 @@ type Unit interface {
 	// every unit does base initialization in `Init`, and this must pass before start running the subtask
 	// other setups can be done in the beginning of `Process`, but this should be treated carefully to make it
 	// compatible with Pause / Resume.
+	//
 	// if initialing successfully, the outer caller should call `Close` when the unit (or the task) finished, stopped or canceled (because other units Init fail).
 	// if initialing fail, Init itself should release resources it acquired before (rolling itself back).
 	Init(ctx context.Context) error