[omniORB] Possible bug?

[renny.koshy at rubixinfotech.com]

I have been trying to use omniORB with VS.NET... don't even ask... we're a
SPARC shop, but of course there's always that one customer... :-)

Anyway... in debug mode, I always get an assertion failure, and decided to
track it down.  Seems that the failure is at line 192 of orboptionsreg.c.

This is because we're calling "isspace()" with an invalid argument, i.e. a
negative number... looking closely I noticed the block:


  p = keybuf;
  while ( isspace(*p) )
    p++;
  key = p;
  if (!*p != '\0') {
    p += strlen(key) - 1;
    while ( isspace((char)(*p)) )
      p--;
    *(++p) = '\0';
  }

Specifically.. should the if block be:

  if (!*p != '\0') {

or

  if (*p != '\0') {

I would think the latter???

I'm going to try it out see what happens...