Merge pull request #65482 from wgliang/master.handle-args

Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Fix 'kubectl cp' with no arguments causes a panic **What this PR does / why we need it**: /kind bug "kubectl cp" with no arguments causes a panic **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: Fixes #65475 **Special notes for your reviewer**: **Release note**: ```release-note Fix 'kubectl cp' with no arguments causes a panic ```

Merge pull request #65482 from wgliang/master.handle-args
0c698a47 · Kubernetes Submit Queue · GitHub · 2a54f83e · 254b02bc · 0c698a47
Unverified Commit 0c698a47 authored Jul 03, 2018 by Kubernetes Submit Queue Committed by GitHub Jul 03, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 0 deletions

cp.go pkg/kubectl/cmd/cp.go +3 -0

No files found.
--- a/pkg/kubectl/cmd/cp.go
+++ b/pkg/kubectl/cmd/cp.go
@@ -164,6 +164,9 @@ func (o *CopyOptions) Validate(cmd *cobra.Command, args []string) error {
 }

 func (o *CopyOptions) Run(args []string) error {
+	if len(args) < 2 {
+		return fmt.Errorf("source and destination are required")
+	}
 	srcSpec, err := extractFileSpec(args[0])
 	if err != nil {
 		return err