Re: Re: Ann: 6.0.3 Release

Chris Tilley <[email protected]> wrote:
> Aengus,Exactly as I said to begin with.
>
> Anyway, As a challenge I have attempted to build this both last night
> and tonight using said 'more efficient method', and it doesn't
> work.Having said that, I just tried your method 5 minutes ago and it
> doesn't work either! Seems to be that this line doesn't work: if
> (strstr(whole, "x64") != NULL) as everything is returning 32-bit.
> FWIW I am using real log data, not stylised info.Just for "fun" I
> dropped a catch all "x64" into the 'other' OS section and ran that
> (knowing that there is a 64-bit IRIX in there. Nada. Unless I'm
> screwing up somewhere further up the chain, the parameter 'whole'
> doesn't contain the data.Chrishttp://camie.dyndns.org/

The whole(!) section of tree.c that breaks down the browser string 
relies on the variable "whole" - line 832, for example.

The original piece that I posted "off the top of my head", refered to 
"x64". The working code that I posted referred to "; x64;"

If you replace lines 868-874 in trees.c with these lines:

       if (strstr(whole, "; x64;") != NULL)
         *name = "Windows:Windows XP64";
       else
         *name = "Windows:Windows XP";
     }
     else if (*(c + 1) == '.' && (*(c + 2) == '2'))
       *name = "Windows:Windows Server 2003";
     else
       *name = "Windows:Unknown Windows";
   }
   else if (*c =='6')  {
     if (strstr(whole, "; x64;") != NULL)
       *name = "Windows:Windows Vista64";
     else
       *name = "Windows:Windows Vista";
     }
     else if (*c >= '7' && *c <= '9')

and recompile, then your new copy of Analog should recognize browser 
strings like these:

Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 6.0; Win64; x64; .NET CLR 
2.0.50727; SLCC1)

Aengus 


+------------------------------------------------------------------------
|  TO UNSUBSCRIBE from this list:
|    http://lists.meer.net/mailman/listinfo/analog-help
|
|  Analog Documentation: http://analog.cx/docs/Readme.html
|  List archives:  http://www.analog.cx/docs/mailing.html#listarchives
|  Usenet version: news://news.gmane.org/gmane.comp.web.analog.general
+------------------------------------------------------------------------