4418 svc.startd crash with a long instance name and more than one dep…

…endency Reviewed by: Yuri Pankov <[email protected]> Reviewed by: Marcel Telka <[email protected]> Reviewed by: Albert Lee <[email protected]> Approved by: Richard Lowe <[email protected]>
illumos · May 27, 2015 · 09f79f7 · 09f79f7
1 parent cb5842f
commit 09f79f7
Showing 1 changed file with 2 additions and 1 deletion.
diff --git 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 */