$OpenBSD$
--- edit.c.orig	Sat Aug 26 13:09:41 1995
+++ edit.c	Mon Feb 14 20:06:18 2011
@@ -143,8 +143,8 @@ static void GetGameValuesPopup (void)
 	ok	   = XtCreateManagedWidget ("ok",    commandWidgetClass, form, args, 1);
 
 	XtSetArg (args[0], XtNfromHoriz, turn_label);
-	if (turn == BLACK) strcpy (value,"black");
-	else  strcpy (value,"white");
+	if (turn == BLACK) strlcpy (value,"black", 15);
+	else  strlcpy (value,"white", 15);
 	turn_switch    = XtCreateManagedWidget (value, commandWidgetClass, form, args, 1);
 
 	XtSetArg (args[0], XtNwidth, &w);
@@ -155,14 +155,14 @@ static void GetGameValuesPopup (void)
 
 	XtSetArg (args[0], XtNfromHoriz, doubler_label);
 	XtSetArg (args[1], XtNfromVert, turn_label);
-	sprintf (value, "%d", doubler.value);
+	snprintf (value, 15, "%d", doubler.value);
 	doubler_switch = XtCreateManagedWidget (value, commandWidgetClass, form, args, 4);
 
 	XtSetArg (args[0], XtNfromHoriz, owner_label);
 	XtSetArg (args[1], XtNfromVert, doubler_label);
-	if (!doubler.owner) strcpy (value,"none");
-	else if (doubler.owner == BLACK)  strcpy (value,"black");
-	else  strcpy (value,"white");
+	if (!doubler.owner) strlcpy (value,"none", 15);
+	else if (doubler.owner == BLACK)  strlcpy (value,"black", 15);
+	else  strlcpy (value,"white", 15);
 	owner_switch   = XtCreateManagedWidget (value, commandWidgetClass, form, args, 4);
 
 {	unsigned short d;
@@ -176,12 +176,12 @@ static void GetGameValuesPopup (void)
 
 	roll[0] =  abs (roll[0]);
 	roll[1] =  abs (roll[1]);
-	sprintf (value, "%d", roll[0]);
+	snprintf (value, 15, "%d", roll[0]);
 	roll_switch[0] = XtCreateManagedWidget (value, commandWidgetClass, form, args, 4);
 
 	XtSetArg (args[0], XtNfromHoriz, roll_switch[0]);
 	XtSetArg (args[1], XtNfromVert, owner_label);
-	sprintf (value, "%d", roll[1]);
+	snprintf (value, 15, "%d", roll[1]);
 	roll_switch[1] = XtCreateManagedWidget (value, commandWidgetClass, form, args, 4);
 
 	XtPopup (popup_shell, XtGrabExclusive);
@@ -204,21 +204,21 @@ static void GetGameValuesPopup (void)
 			else if (event.xbutton.window == XtWindow (doubler_switch)) {
 				doubler.value *= 2;
 				if (doubler.value > 64) doubler.value = 1;
-				sprintf (value, "%d", doubler.value);
+				snprintf (value, 15, "%d", doubler.value);
 				XtSetArg (args[0], XtNlabel, value);
 				XtSetValues (doubler_switch, args, 1);
 			}
 			else if (event.xbutton.window == XtWindow (owner_switch)) {
 				if (!doubler.owner) {
-					strcpy (value, "black");
+					strlcpy (value, "black", 15);
 					doubler.owner = BLACK;
 				}
 				else if (doubler.owner == BLACK) {
-					strcpy (value, "white");
+					strlcpy (value, "white", 15);
 					doubler.owner = WHITE;
 				}
 				else  {
-					strcpy (value, "none");
+					strlcpy (value, "none", 15);
 					doubler.owner = 0;
 				}
 				XtSetArg (args[0], XtNlabel, value);
@@ -227,11 +227,11 @@ static void GetGameValuesPopup (void)
 			else if (event.xbutton.window == XtWindow (roll_label)) {
 				if (!ignore_dice) {
 					ignore_dice = 1;
-					strcpy (value, "ignore roll   ");
+					strlcpy (value, "ignore roll   ", 15);
 				}
 				else {
 					ignore_dice = 0;
-					strcpy (value, "dice:         ");
+					strlcpy (value, "dice:         ", 15);
 				}
 				XtSetArg (args[0], XtNlabel, value);
 				XtSetValues (roll_label, args, 1);
@@ -239,14 +239,14 @@ static void GetGameValuesPopup (void)
 			else if (event.xbutton.window == XtWindow (roll_switch[0])) {
 				roll[0] ++;
 				if (roll[0] == 7) roll[0] = 1;
-				sprintf (value, "%d", roll[0]);
+				snprintf (value, 15, "%d", roll[0]);
 				XtSetArg (args[0], XtNlabel, value);
 				XtSetValues (roll_switch[0], args, 1);
 			}
 			else if (event.xbutton.window == XtWindow (roll_switch[1])) {
 				roll[1] ++;
 				if (roll[1] == 7) roll[1] = 1;
-				sprintf (value, "%d", roll[1]);
+				snprintf (value, 15, "%d", roll[1]);
 				XtSetArg (args[0], XtNlabel, value);
 				XtSetValues (roll_switch[1], args, 1);
 			}
