[FAB-14491] stop using Sprintf in Errorf

fmt.Errorf formats a message with args; no need to use fmt.Sprintf
inside of that.

Change-Id: I6d787607c14432c710fd197acca2c141e02bc2bb
Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>

Author: sykesm

core/container/inproccontroller/inproccontroller.go

@@ -198,19 +198,19 @@ func (ipc *inprocContainer) launchInProc(id string, args []string, env []string)
 func (vm *InprocVM) Start(ccid ccintf.CCID, args []string, env []string, filesToUpload map[string][]byte, builder container.Builder) error {
 	ipctemplate := vm.registry.getType(ccid)
 	if ipctemplate == nil {
-		return fmt.Errorf(fmt.Sprintf("%s not registered", ccid))
+		return fmt.Errorf("%s not registered", ccid)
 	}
 
 	instName := vm.GetVMName(ccid)
 
 	ipc, err := vm.getInstance(ipctemplate, instName, args, env)
 
 	if err != nil {
-		return fmt.Errorf(fmt.Sprintf("could not create instance for %s", instName))
+		return fmt.Errorf("could not create instance for %s", instName)
 	}
 
 	if ipc.running {
-		return fmt.Errorf(fmt.Sprintf("chaincode running %s", ccid))
+		return fmt.Errorf("chaincode running %s", ccid)
 	}
 
 	ipc.running = true