From jhs@berklix.org  Thu May 15 14:15:23 2008
Return-Path: <jhs@berklix.org>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 84A7F106564A
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 15 May 2008 14:15:23 +0000 (UTC)
	(envelope-from jhs@berklix.org)
Received: from flat.berklix.org (flat.berklix.org [83.236.223.115])
	by mx1.freebsd.org (Postfix) with ESMTP id E04C28FC13
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 15 May 2008 14:15:22 +0000 (UTC)
	(envelope-from jhs@berklix.org)
Received: from js.berklix.net (p549A7E88.dip.t-dialin.net [84.154.126.136])
	(authenticated bits=0)
	by flat.berklix.org (8.13.8/8.13.8) with ESMTP id m4FDb7OB014239
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 15 May 2008 15:37:08 +0200 (CEST)
	(envelope-from jhs@berklix.org)
Received: from fire.js.berklix.net (fire.js.berklix.net [192.168.91.41])
	by js.berklix.net (8.13.8/8.13.8) with ESMTP id m4FDgQUS005881
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 15 May 2008 15:42:27 +0200 (CEST)
	(envelope-from jhs@berklix.org)
Received: from fire.js.berklix.net (localhost [127.0.0.1])
	by fire.js.berklix.net (8.13.8/8.13.8) with ESMTP id m4FDgGTQ096136
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 15 May 2008 15:42:21 +0200 (CEST)
	(envelope-from jhs@fire.js.berklix.net)
Received: (from jhs@localhost)
	by fire.js.berklix.net (8.13.8/8.13.8/Submit) id m4FDgBhn096135;
	Thu, 15 May 2008 15:42:11 +0200 (CEST)
	(envelope-from jhs)
Message-Id: <200805151342.m4FDgBhn096135@fire.js.berklix.net>
Date: Thu, 15 May 2008 15:42:11 +0200 (CEST)
From: "\"Julian H. Stacey\" <jhs@berklix.com>" <jhs@berklix.org>
Reply-To: "\"Julian H. Stacey\" <jhs@berklix.com>" <jhs@berklix.org>
To: FreeBSD-gnats-submit@freebsd.org
Subject: qualify bad diagnostic in src/usr.sbin/timed/timedc/cmds.c
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         123703
>Category:       bin
>Synopsis:       [patch] timed(8): qualify bad diagnostic in src/usr.sbin/timed/timedc/cmds.c
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Thu May 15 14:20:01 UTC 2008
>Closed-Date:    
>Last-Modified:  Sun May 18 23:41:19 UTC 2008
>Originator:     "Julian H. Stacey" <jhs@berklix.com>
>Release:        FreeBSD 7.0-RELEASE i386
>Organization:
http://berklix.com BSD Linux Unix Consultancy, Munich/Muenchen.
>Environment:
System: FreeBSD fire.js.berklix.net 6.2-RELEASE FreeBSD 6.2-RELEASE #0: Mon Sep 17 23:01:21 CEST 2007 jhs@fire.js.berklix.net:/usr1/src/sys/amd64/compile/FIRE64.small amd64


	
>Description:
	qualify bad diagnostic in src/usr.sbin/timed/timedc/cmds.c
>How-To-Repeat:
	get remote host as timed master,
	localhost as timed slave,
	then kill remote
	& see local lieing.
>Fix:

	
Copy appended of 
http://berklix.com/~jhs/src/bsd/fixes/freebsd/src/gen/usr.sbin/timed/timed/cmds.c.diff.REL=ALL

Before this patch if the remote master was killed, the local slave lied & said
	communication error with localhost
With this patch the code at least no longer lies.

Patch is hack, 
I would like to read the code & do better, but I dont have time.

the code has not been unchanged for over 4 years 
this diff also applies to 7.0-RELEASE

*** 6.2-RELEASE/src/usr.sbin/timed/timedc/cmds.c	Wed Feb  4 22:59:29 2004
--- new-generic/src/usr.sbin/timed/timedc/cmds.c	Thu May 15 15:20:19 2008
***************
*** 356,361 ****
--- 356,364 ----
  			}
  		} else {
  			printf("communication error with %s\n", tgtname);
+ 			printf(
+ " (A dead remote master is also reported as \"... with localhost\").\n"
+ 				);
  		}
  	} while (++i < argc);
  }


>Release-Note:
>Audit-Trail:
>Unformatted:
