Skip to content

Commit 079817c

Browse files
Merge pull request #1926 from Ace-Tang/fix_spec_proc
libcontainer: fix potential panic if spec.Process is nil
2 parents b1068fb + 16d55f1 commit 079817c

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

libcontainer/specconv/spec_linux.go

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -251,19 +251,19 @@ func CreateLibcontainerConfig(opts *CreateOpts) (*configs.Config, error) {
251251
}
252252
}
253253
}
254-
if spec.Process.SelinuxLabel != "" {
255-
config.ProcessLabel = spec.Process.SelinuxLabel
256-
}
257254
if spec.Process != nil {
258255
config.OomScoreAdj = spec.Process.OOMScoreAdj
259-
}
260-
if spec.Process.Capabilities != nil {
261-
config.Capabilities = &configs.Capabilities{
262-
Bounding: spec.Process.Capabilities.Bounding,
263-
Effective: spec.Process.Capabilities.Effective,
264-
Permitted: spec.Process.Capabilities.Permitted,
265-
Inheritable: spec.Process.Capabilities.Inheritable,
266-
Ambient: spec.Process.Capabilities.Ambient,
256+
if spec.Process.SelinuxLabel != "" {
257+
config.ProcessLabel = spec.Process.SelinuxLabel
258+
}
259+
if spec.Process.Capabilities != nil {
260+
config.Capabilities = &configs.Capabilities{
261+
Bounding: spec.Process.Capabilities.Bounding,
262+
Effective: spec.Process.Capabilities.Effective,
263+
Permitted: spec.Process.Capabilities.Permitted,
264+
Inheritable: spec.Process.Capabilities.Inheritable,
265+
Ambient: spec.Process.Capabilities.Ambient,
266+
}
267267
}
268268
}
269269
createHooks(spec, config)

0 commit comments

Comments
 (0)