Bug found in Arla CVS

  This message is in MIME format.  The first part should be readable text,
  while the remaining parts are likely unreadable without MIME-aware tools.

---276651502-256313640-1453479862=:14802
Content-Type: TEXT/PLAIN; format=flowed; charset=US-ASCII


Hello,

when testing Arla I found a bug in the CVS version that I thought I should 
warn you about:

In the CVS version they changed from uppercase to lowercase file and dir 
names in the cache:

2c2
<  * Copyright (c) 2005-2006, Stockholms Universitet
---
>  * Copyright (c) 2005-2007, Stockholms Universitet
34c34
< /* $Id: nnpfs_blocks.h,v 1.2 2006/10/24 16:33:31 tol Exp $ */
---
> /* $Id: nnpfs_blocks.h,v 1.3 2007/01/24 17:09:17 tol Exp $ */
39,41c39,45
< #define NNPFS_CACHE_FILE_PATH    "%02X/%02X/%02llX"
< #define NNPFS_CACHE_DIR_PATH     "%02X/%02X@"
< #define NNPFS_CACHE_PATH_SIZE    ( 6 +1 +2+1 +16 +1)
---
> #define NNPFS_CACHE_FILE_DIR1        "%02x"
> #define NNPFS_CACHE_FILE_DIR_PATH    NNPFS_CACHE_FILE_DIR1 "/" NNPFS_CACHE_FILE_DIR1
> #define NNPFS_CACHE_FILE_BLOCK_PATH  "%02llx"
> 
> #define NNPFS_CACHE_FILE_PATH   NNPFS_CACHE_FILE_DIR_PATH "/" NNPFS_CACHE_FILE_BLOCK_PATH
> #define NNPFS_CACHE_DIR_PATH    NNPFS_CACHE_FILE_DIR_PATH "@"
> #define NNPFS_CACHE_PATH_SIZE   ( 6 +1 +2+1 +16 +1 )

However the old look was hardcoded in some places in arlad/fcache.c, here 
is a patch that corrects this: (Also remember this when mixing different 
versions of arlad and nnpfs)

--- src/arla-0.90/arlad/fcache.c	2006-12-11 17:23:24.000000000 +0100
+++ src/arla-0.90-new/arlad/fcache.c	2016-01-22 17:00:43.000000000 +0100
@@ -713,7 +713,7 @@ fcache_have_wanted(FCacheEntry *entry, u
  int
  fcache_dir_name (FCacheEntry *entry, char *s, size_t len)
  {
-    return snprintf (s, len, "%02X", entry->index / 0x100);
+    return snprintf (s, len, NNPFS_CACHE_FILE_DIR1, entry->index / 0x100);
  }

  /*
@@ -723,7 +723,7 @@ fcache_dir_name (FCacheEntry *entry, cha
  static int
  fcache_file_name (FCacheEntry *entry, char *s, size_t len)
  {
-    return snprintf (s, len, "%02X/%02X",
+    return snprintf (s, len, NNPFS_CACHE_FILE_DIR_PATH,
  		     entry->index / 0x100, entry->index % 0x100);
  }

@@ -815,7 +815,7 @@ fcache_fhget (char *filename, fcache_cac
  	int ret, a, b;
  	char buf[1024];

-	ret = sscanf(filename, "%02X/%02X", &a, &b);
+	ret = sscanf(filename, NNPFS_CACHE_FILE_DIR_PATH, &a, &b);
  	if (ret != 2)
  	    return EINVAL;

@@ -824,7 +824,7 @@ fcache_fhget (char *filename, fcache_cac

  	ret = snprintf((char *)handle->nnpfs_handle,
  		       sizeof(handle->nnpfs_handle),
-		       "%s\\%02X\\%02X", buf, a, b);
+		       "%s\\" NNPFS_CACHE_FILE_DIR_PATH, buf, a, b);

  	if (ret > 0 && ret < sizeof(handle->nnpfs_handle))
  	    handle->valid = 1;
---276651502-256313640-1453479862=:14802
Content-Type: TEXT/PLAIN; charset=US-ASCII; name=patch-arla-fcache
Content-Transfer-Encoding: BASE64
Content-ID: <[email protected]>
Content-Description: 
Content-Disposition: attachment; filename=patch-arla-fcache

LS0tIHNyYy9hcmxhLTAuOTAvYXJsYWQvZmNhY2hlLmMJMjAwNi0xMi0xMSAx
NzoyMzoyNC4wMDAwMDAwMDAgKzAxMDANCisrKyBzcmMvYXJsYS0wLjkwLW5l
dy9hcmxhZC9mY2FjaGUuYwkyMDE2LTAxLTIyIDE3OjAwOjQzLjAwMDAwMDAw
MCArMDEwMA0KQEAgLTcxMyw3ICs3MTMsNyBAQCBmY2FjaGVfaGF2ZV93YW50
ZWQoRkNhY2hlRW50cnkgKmVudHJ5LCB1DQogaW50DQogZmNhY2hlX2Rpcl9u
YW1lIChGQ2FjaGVFbnRyeSAqZW50cnksIGNoYXIgKnMsIHNpemVfdCBsZW4p
DQogew0KLSAgICByZXR1cm4gc25wcmludGYgKHMsIGxlbiwgIiUwMlgiLCBl
bnRyeS0+aW5kZXggLyAweDEwMCk7DQorICAgIHJldHVybiBzbnByaW50ZiAo
cywgbGVuLCBOTlBGU19DQUNIRV9GSUxFX0RJUjEsIGVudHJ5LT5pbmRleCAv
IDB4MTAwKTsNCiB9DQogDQogLyoNCkBAIC03MjMsNyArNzIzLDcgQEAgZmNh
Y2hlX2Rpcl9uYW1lIChGQ2FjaGVFbnRyeSAqZW50cnksIGNoYQ0KIHN0YXRp
YyBpbnQNCiBmY2FjaGVfZmlsZV9uYW1lIChGQ2FjaGVFbnRyeSAqZW50cnks
IGNoYXIgKnMsIHNpemVfdCBsZW4pDQogew0KLSAgICByZXR1cm4gc25wcmlu
dGYgKHMsIGxlbiwgIiUwMlgvJTAyWCIsDQorICAgIHJldHVybiBzbnByaW50
ZiAocywgbGVuLCBOTlBGU19DQUNIRV9GSUxFX0RJUl9QQVRILA0KIAkJICAg
ICBlbnRyeS0+aW5kZXggLyAweDEwMCwgZW50cnktPmluZGV4ICUgMHgxMDAp
Ow0KIH0NCiANCkBAIC04MTUsNyArODE1LDcgQEAgZmNhY2hlX2ZoZ2V0IChj
aGFyICpmaWxlbmFtZSwgZmNhY2hlX2NhYw0KIAlpbnQgcmV0LCBhLCBiOw0K
IAljaGFyIGJ1ZlsxMDI0XTsNCiANCi0JcmV0ID0gc3NjYW5mKGZpbGVuYW1l
LCAiJTAyWC8lMDJYIiwgJmEsICZiKTsNCisJcmV0ID0gc3NjYW5mKGZpbGVu
YW1lLCBOTlBGU19DQUNIRV9GSUxFX0RJUl9QQVRILCAmYSwgJmIpOw0KIAlp
ZiAocmV0ICE9IDIpDQogCSAgICByZXR1cm4gRUlOVkFMOw0KIA0KQEAgLTgy
NCw3ICs4MjQsNyBAQCBmY2FjaGVfZmhnZXQgKGNoYXIgKmZpbGVuYW1lLCBm
Y2FjaGVfY2FjDQogCQ0KIAlyZXQgPSBzbnByaW50ZigoY2hhciAqKWhhbmRs
ZS0+bm5wZnNfaGFuZGxlLCANCiAJCSAgICAgICBzaXplb2YoaGFuZGxlLT5u
bnBmc19oYW5kbGUpLA0KLQkJICAgICAgICIlc1xcJTAyWFxcJTAyWCIsIGJ1
ZiwgYSwgYik7DQorCQkgICAgICAgIiVzXFwiIE5OUEZTX0NBQ0hFX0ZJTEVf
RElSX1BBVEgsIGJ1ZiwgYSwgYik7DQogDQogCWlmIChyZXQgPiAwICYmIHJl
dCA8IHNpemVvZihoYW5kbGUtPm5ucGZzX2hhbmRsZSkpDQogCSAgICBoYW5k
bGUtPnZhbGlkID0gMTsNCg==

---276651502-256313640-1453479862=:14802
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
Arla-drinkers mailing list
[email protected]
https://lists.stacken.kth.se/mailman/listinfo/arla-drinkers

---276651502-256313640-1453479862=:14802--