nil pointer dereference when shutting down cells

[skozlov404]

It seems like the logic in the code below is wrong. When (process == nil || len(process.Services) == 0), but at the same time len(childrenProcesses) != 0 - the code doesn't return nil, but instead continuing to log.Info, where process == nil is deferenced.

cells/cmd/start.go

Lines 288 to 292 in e36a254

	childrenProcesses := registry.Default.GetCurrentChildrenProcesses()
	if (process == nil || len(process.Services) == 0) && len(childrenProcesses) == 0 {
	return nil
	}
	log.Info("Services are still running ", zap.Any("services", process.Services))

[cdujeu]

Hi @skozlov404 , thanks for reporting. @ghecquet can you have a look?