Re: squid-2 acl fileext

Hi Henrik:

I think I'm messing something up.
Well, I'm trying to store filename extensions supplied by the
user to a wordlist bound to 'data' member of 'acl *', so I did:

1) at 'aclParseAclLine()' is called:

    case ACL_FILEEXT:
        aclParseFileExt(&A->data);
        break;

2) at 'aclParseFileExt()' (it's a test implementation):

void
aclParseFileExt(void *curlist)
{
  char *t = NULL;

  while ((t = strtokFile())) {
    if (*t != '.') {
      debug (28,0) ("aclParseFileExt: Invalid filename extension '%s'\n",t);
      continue;
    }
    if (strlen (t) > 8) { /* 8 will be a macro */
      debug (28,0) ("aclParseFileExt: Filename extension too long '%s'\n",t);
      continue;
    }
    debug (28,3) ("aclParseFileExt: added '%s' extension\n",
wordlistAdd (curlist, t));
  }
}

My problem is that I'm not be able to access A->data (the wordlist) in an
'aclMatchFileExt()' (it's also a test implementation):

int
aclMatchFileExt(void *dataptr, const char *url)
{
  char *e = NULL;
  wordlist *l = NULL;
  int i = 0;

  l = dataptr;

  while (l) {
    debug (28,3) ("aclMatchExt: extension = '%s'\n", l->key);
    l = l->next;
  }

  return 0;
}

Any Help?

thanks in advance
Lucas Brasilino

2007/7/20, Henrik Nordstrom <henrik@henriknordstrom.net>:
> On ons, 2007-07-25 at 12:41 +0200, Emilio Casbas wrote:
>
> > Is mandatory to do a cvs checkout with working code? or, would it be
> > possible to do it with incomplete code?
>
> It's free to break branches whenever the developer working on the branch
> sees fit.
>
> Personally I prefer to commit the changes often, like once per hour or
> so. Even if things isn't working. It's easy to undo bad changes later if
> needed.
>
> Regards
> Henrik
>
>
>
Received on Fri Jul 20 2007 - 14:37:44 MDT