-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Martin Preuss schrieb:
> [...]
>> I think we could do it more easily: At that particular line we could just
>> remove the escaping, it isn't necessary at that point. I don't even know
>> why I put it there in the first place...
> [...]
>
> Ok, I just modified AqHBCI as noted above. It still works on my Linux box and
> it should now work on Windows as well...
Ok, I have to admit you were right and I was wrong: The
GWEN_Path_Convert call in question actually wasn't the point where the
"C:" escaping happened. I saw the erroneous path still being created
even with SVN r1186, so eventually I ran this with
GWEN_LOGLEVEL="verbous" and there it is:
5:2007/02/19 15-39-27:aqhbci(1292):job.c: 1575: Found a bank message
## some own printf indicate we are entering AH_HBCI_SaveSettings() now
7:2007/02/19 15-39-27:gwen(1292):directory_all.c: 116: Checking path "C%3A"
7:2007/02/19 15-39-27:gwen(1292):directory_all.c: 119: stat: No such
file or directory (C%3A)
7:2007/02/19 15-39-27:gwen(1292):directory_all.c: 157: Entry "C%3A"
does not exist
7:2007/02/19 15-39-27:gwen(1292):directory_all.c: 193: Creating folder
"C%3A"
(...)
And so on. So the problem was the GWEN_Directory_GetPath at the
beginning of AH_HBCI_SaveSettings, which had the flags
GWEN_PATH_FLAGS_ESCAPE | GWEN_PATH_FLAGS_TOLERANT_ESCAPE set. Removing
these escape flags will remove the spurious directory creation, see
attached patch. With this, I now get this:
5:2007/02/19 15-50-19:aqhbci(1908):job.c: 1575: Found a bank message
7:2007/02/19 15-50-19:gwen(1908):directory_all.c: 116: Checking path "C:\"
7:2007/02/19 15-50-19:gwen(1908):directory_all.c: 129: Checking for type
7:2007/02/19 15-50-19:gwen(1908):directory_all.c: 212: Entry "C:\" exists
7:2007/02/19 15-50-19:gwen(1908):directory_all.c: 214: Returning this: C:\
Is the patch ok for you?
By the way, AH_HBCI_LoadSettings isn't used anymore anywhere. If you
want to, you could remove it altogether.
Regards,
Christian
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.1 (MingW32)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iQCVAwUBRdm7bGXAi+BfhivFAQIlywP+KaC1+aTOwkt5UCewp0IxdFwQK6uWEdr4
43ZxGG9RuXmOORR3qMpVwXkqeu2Cfa6i2TqChlgoG6Rw3f1WdqDXGAGRwgUjo4mD
33rP/+4+8jq4gSUh8javF/7Gj0a7Z7PHUH+pik7KeXRQNG7H7hvr+0/UG7Z5h13H
hQ93bPg8YrQ=
=1/ho
-----END PGP SIGNATURE-----
Index: msglayer/hbci.c
===================================================================
RCS file:
/cvsroot/aqbanking/aqbanking/src/plugins/backends/aqhbci/plugin/msglayer/hbci.c,v
retrieving revision 1.13
diff -u -r1.13 hbci.c
--- msglayer/hbci.c 19 Feb 2007 09:24:04 -0000 1.13
+++ msglayer/hbci.c 19 Feb 2007 14:51:41 -0000
@@ -860,8 +860,6 @@
int AH_HBCI_SaveSettings(const char *path, GWEN_DB_NODE *db){
/* check for existence of that file */
if (GWEN_Directory_GetPath(path,
- GWEN_PATH_FLAGS_ESCAPE |
- GWEN_PATH_FLAGS_TOLERANT_ESCAPE |
GWEN_PATH_FLAGS_VARIABLE)) {
DBG_INFO(AQHBCI_LOGDOMAIN, "Path \"%s\" is not available", path);
return -1;
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Aqbanking-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/aqbanking-devel
