Bug#163117: closed by Mart??n Ferrari <tincho at debian.org> (Closing old bug)

[Jeff Goeke-Smith]

At this point, I haven't actively worked on the ricochet hardware in
10 years.  If there's a problem with it working, it will be exploring
a whole new set of problems at that time.

Thanks for tracking it this long.

On Mon, May 12, 2014 at 01:51:18AM +0000, Debian Bug Tracking System wrote:
> This is an automatic notification regarding your Bug report
> which was filed against the net-tools package:
> 
> #163117: net-tools doesn't support newer 128Kbps ricochet modems
> 
> It has been closed by Mart??n Ferrari <tincho at debian.org>.
> 
> Their explanation is attached below along with your original report.
> If this explanation is unsatisfactory and you have not received a
> better one in a separate message then please contact Mart??n Ferrari <tincho at debian.org> by
> replying to this email.
> 
> 
> -- 
> 163117: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=163117
> Debian Bug Tracking System
> Contact owner at bugs.debian.org with problems

> From: Mart?n Ferrari <tincho at debian.org>
> Date: Mon, 12 May 2014 03:49:01 +0200
> To: 163117-done at bugs.debian.org
> Subject: Closing old bug
> 
> There was no response to my request from 7 years ago, so I think this
> can be closed now :)
> 
> -- 
> Mart?n Ferrari (Tincho)
> 



> From: "Jeff Goeke-Smith" <goekesmi at msu.edu>
> Date: Wed, 02 Oct 2002 15:10:24 -0400
> To: "Debian Bug Tracking System" <submit at bugs.debian.org>
> Subject: net-tools doesn't support newer 128Kbps ricochet modems
> 
> Package: net-tools
> Version: 1.60-4
> Severity: normal
> Tags: patch upstream
> 
> The net-tools package doesn't support the 128Kbps ricochet modems when
> in strip mode.  With the addition of the GS, GT, and Merlin cards for
> Ricochet, the hardware address format changed.  The included patch
> updates the strip.c file in net-tools/lib/ so as to appropriately 
> support the longer hardware addresses.  An associated kernel patch
> is also necessary for support of the modern ricochet modems.  This
> patch appears to remain backwards compatable with older ricochet
> modems that have the original hardware address format.
> 
> The included patch was derived from the patches included in section 8.4
> of http://phobos.illtel.denver.co.us/~abelits/metricom/ .  The patch
> located there was for net-tools-1.57.  The patch for 1.57 didn't apply
> automatically, so I have hand applied this patch to the 1.60 tree and 
> confirmed that it works.
> 
> Patch follows.
> 
> --- strip.c.orig	2002-09-26 21:17:36.000000000 -0400
> +++ strip.c	2002-09-26 21:43:49.000000000 -0400
> @@ -42,19 +42,30 @@
>  pr_strip(unsigned char *ptr)
>  {
>    static char buff[64];
> -  if(ptr[1])
> -      sprintf(buff, "%02x-%02x%02x-%02x%02x", *(ptr+1), *(ptr+2), *(ptr+3),
> -          *(ptr+4), *(ptr+5));
> -   else
> +  if(*ptr)
> +  {
> +    sprintf(buff, "%01x%02x-%02x%02x-%02x%02x", ((*ptr) & 0xF) ^ 0xF,
> +           (*(ptr+1)) ^ 0xFF, *(ptr+2), *(ptr+3),
> +           *(ptr+4), *(ptr+5));
> +  }else{
> +    if(ptr[1])
> +    {
> +      sprintf(buff, "%02x-%02x%02x-%02x%02x", (*(ptr+1)) ^ 0xFF, *(ptr+2),
> +             *(ptr+3),
> +             *(ptr+4), *(ptr+5));
> +    }else{
>        sprintf(buff, "%02x%02x-%02x%02x", *(ptr+2), *(ptr+3), *(ptr+4),
> -	  *(ptr+5));
> -  return buff;
> +             *(ptr+5));
> +    }
> +  }
> +return buff;
>  }
>  
>  static int
>  in_strip(char *bufp, struct sockaddr *sap)
>  {
>    int i,i0;
> +  char tempbuf[2];
>    MetricomAddress *haddr = (MetricomAddress *) (sap->sa_data);
>  
>  
> @@ -69,15 +80,26 @@
>    if (bufp[i] != '-')
>      return -1;
>  
> -  if(i-i0 == 2)
> +  if(i-i0 == 3)
>    {
> -     haddr->c[1] = strtol(&bufp[i0], 0, 16);
> -     i++;
> -     if(bufp[i] == 0) return -1;
> +    tempbuf[0] = bufp[i0];
> +    tempbuf[1] = 0;
> +    haddr->c[0] = (strtol(tempbuf, 0, 16)) ^ 0xF;
> +    haddr->c[1] = (strtol(&bufp[i0+1], 0, 16)) ^ 0xFF;
> +    i++;
> +    if(bufp[i] == 0) return -1;
>    }else{
> -     haddr->c[1] = 0;
> -     i=i0;
> +    if(i-i0 == 2)
> +    {
> +      haddr->c[1] = (strtol(&bufp[i0], 0, 16)) ^ 0xFF;
> +      i++;
> +      if(bufp[i] == 0) return -1;
> +    }else{
> +      haddr->c[1] = 0;
> +      i=i0;
> +    }
>    }
> +
>    haddr->c[2] = strtol(&bufp[i], 0, 16) >> 8;
>    haddr->c[3] = strtol(&bufp[i], 0, 16) & 0xFF;
>  
> 
> 
> 
> -- System Information:
> Debian Release: testing/unstable
> Architecture: i386
> Kernel: Linux phobos 2.4.19 #9 Sat Sep 14 14:22:53 EDT 2002 i686
> Locale: LANG=C, LC_CTYPE=C
> 
> Versions of packages net-tools depends on:
> ii  libc6                         2.2.5-14.3 GNU C Library: Shared libraries an
> 
> -- no debconf information
> 
> 


-- 
Jeff Goeke-Smith
goekesmi at msu.edu
Infrastructure Design
Michigan State University