From 09f79f7c66b85f056db11f58210dc6182c9b1aef Mon Sep 17 00:00:00 2001 From: Dan Vatca Date: Wed, 27 May 2015 13:29:55 +0300 Subject: [PATCH] 4418 svc.startd crash with a long instance name and more than one dependency Reviewed by: Yuri Pankov Reviewed by: Marcel Telka Reviewed by: Albert Lee Approved by: Richard Lowe --- usr/src/cmd/svc/startd/graph.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/usr/src/cmd/svc/startd/graph.c b/usr/src/cmd/svc/startd/graph.c index 7fbf17a6ec44..c831c9930158 100644 --- a/usr/src/cmd/svc/startd/graph.c +++ b/usr/src/cmd/svc/startd/graph.c @@ -21,6 +21,7 @@ /* * Copyright (c) 2004, 2010, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2015, Syneto S.R.L. All rights reserved. */ /* @@ -2588,7 +2589,7 @@ process_dependency_pg(scf_propertygroup_t *pg, struct deppg_info *info) fmri_sz = strlen(info->v->gv_name) + 1 + len + 1; fmri = startd_alloc(fmri_sz); - (void) snprintf(fmri, max_scf_name_size, "%s>%s", info->v->gv_name, + (void) snprintf(fmri, fmri_sz, "%s>%s", info->v->gv_name, pg_name); /* Validate the pg before modifying the graph */