diff --git a/components/engine/libcontainerd/client_windows.go b/components/engine/libcontainerd/client_windows.go
index 78b3b992f1..484f465228 100644
--- a/components/engine/libcontainerd/client_windows.go
+++ b/components/engine/libcontainerd/client_windows.go
@@ -295,9 +295,8 @@ func (clnt *client) Signal(containerID string, sig int) error {
 	} else {
 		// Terminate Process
 		if err := cont.hcsProcess.Kill(); err != nil {
+			// ignore errors
 			logrus.Warnf("Failed to terminate pid %d in %s: %q", cont.systemPid, containerID, err)
-			// Ignore errors
-			err = nil
 		}
 	}
 
diff --git a/components/engine/plugin/backend.go b/components/engine/plugin/backend.go
index 1b6daa4770..8dcca58cb3 100644
--- a/components/engine/plugin/backend.go
+++ b/components/engine/plugin/backend.go
@@ -113,7 +113,7 @@ func (pm *Manager) Push(name string, metaHeader http.Header, authConfig *types.A
 	_, err = distribution.Push(name, pm.registryService, metaHeader, authConfig, config, rootfs)
 	// XXX: Ignore returning digest for now.
 	// Since digest needs to be written to the ProgressWriter.
-	return nil
+	return err
 }
 
 // Remove deletes plugin's root directory.