Re: COSS problem

From: Guido Serassio <guido.serassio@dont-contact.us>
Date: Sat, 23 Sep 2006 11:14:57 +0200

Hi Adrian,

Il 03.44 23/09/2006 Adrian Chadd ha scritto:
> >
> > The attached patch adds a stripePath() function similar to the 3.0 one.
> > When the coss cache_dir argument is a directory, the "/stripe" suffix
> > is automatically added.
>
>Looks good!

OK, I will commit.

Another thing:

In storeCossDirSwapLogFile(), if Config.Log.swap is given, things are
always OK, but if Config.Log.swap is not configured and sd->path is
not a directory, aka is a regular file or a device, things are very
wrong: we append "/swap.state" to sd->path ....

I think that here we must also add a check if sd->path is a
directory, and squid should fatal() if a device or file is specified
without configuring cache_swap_log in squid.conf.

Regards

Guido

-
========================================================
Guido Serassio
Acme Consulting S.r.l. - Microsoft Certified Partner
Via Lucia Savarino, 1 10098 - Rivoli (TO) - ITALY
Tel. : +39.011.9530135 Fax. : +39.011.9781115
Email: guido.serassio@acmeconsulting.it
WWW: http://www.acmeconsulting.it/
Received on Sat Sep 23 2006 - 03:15:19 MDT

This archive was generated by hypermail pre-2.1.9 : Sun Oct 01 2006 - 12:00:06 MDT