diff --git a/hrp/pkg/uixt/action.go b/hrp/pkg/uixt/action.go
index 7fbb960b..27e96ed8 100644
--- a/hrp/pkg/uixt/action.go
+++ b/hrp/pkg/uixt/action.go
@@ -385,7 +385,7 @@ func (dExt *DriverExt) GenAbsScope(x1, y1, x2, y2 float64) AbsScope {
 	return AbsScope{absX1, absY1, absX2, absY2}
 }
 
-func (dExt *DriverExt) DoAction(action MobileAction) error {
+func (dExt *DriverExt) DoAction(action MobileAction) (err error) {
 	log.Debug().
 		Str("method", string(action.Method)).
 		Interface("params", action.Params).
@@ -393,11 +393,19 @@ func (dExt *DriverExt) DoAction(action MobileAction) error {
 	actionStartTime := time.Now()
 
 	defer func() {
-		log.Debug().
-			Str("method", string(action.Method)).
-			Interface("params", action.Params).
-			Int64("elapsed(ms)", time.Since(actionStartTime).Milliseconds()).
-			Msg("uixt action end")
+		if err != nil {
+			log.Error().Err(err).
+				Str("method", string(action.Method)).
+				Interface("params", action.Params).
+				Int64("elapsed(ms)", time.Since(actionStartTime).Milliseconds()).
+				Msg("uixt action end")
+		} else {
+			log.Debug().
+				Str("method", string(action.Method)).
+				Interface("params", action.Params).
+				Int64("elapsed(ms)", time.Since(actionStartTime).Milliseconds()).
+				Msg("uixt action end")
+		}
 	}()
 
 	switch action.Method {
diff --git a/hrp/step_mobile_ui.go b/hrp/step_mobile_ui.go
index 1b8cb4f0..c90c88c8 100644
--- a/hrp/step_mobile_ui.go
+++ b/hrp/step_mobile_ui.go
@@ -604,8 +604,8 @@ func runStepMobileUI(s *SessionRunner, step *TStep) (stepResult *StepResult, err
 		}
 
 		// automatic handling of pop-up windows on each step finished
-		if err = uiDriver.AutoPopupHandler(); err != nil {
-			log.Error().Err(err).Str("step", step.Name).Msg("auto handle popup failed")
+		if err2 := uiDriver.AutoPopupHandler(); err2 != nil {
+			log.Error().Err(err2).Str("step", step.Name).Msg("auto handle popup failed")
 		}
 
 		// save attachments