poinst fails on large number of files

[frankie]

Hello Pelle,
Poinst fails with all commands involving multiple files as 'Files' or 'Delete'.
When the files bundle is greater that 478 units Poinst blows-up. 
Took me a while to understand where the problem is, but at the end I got it.

[Pelle]

Sorry, but I really don't understand. Can you give me more info?

[frankie]

Hi Pelle,
in a statement like:

Code: [Select]

File *.*

; or

Delete *.*
when the number of files is more than 478 the poinst crashes.

Some more details. Consider the snippet to clean include directory after installing fSDK:

Code: [Select]

Section Uninstall
    ...
    ; removefiles
Delete "$INSTDIR\fSDK.txt"
MessageBox MB_YESNO|MB_ICONQUESTION "Remove all headers ?" IDNO dontdeleteheaders
;Delete "$INSTDIR\Include\Win\*.*"
!Include "DeleteHeaders.pis"
dontdeleteheaders:
     ...
The file DeleteHeaders.pis, attached, contains the list of more than 1300 files from fSDK. Commenting out the Delete statement from 478th file on make poinst work. Uncomment even one more file and it crashes.

[Pelle]

Code: [Select]

File *.*


This will collect files into the setup. The biggest test I can find right now is 556 files, which works fine here.

Code: [Select]

Delete *.*


This will just remember the search pattern in the setup, the files are removed during runtime (by Windows API).

Two completely different operations stuck on 478? There must be more to this story than this...

Shot in the dark, but have you checked your disk for faults? Memory?

[TimoVJL]

A small test:

Code: [Select]

Name "Example3"
OutFile "test1.exe"
InstallDir Test1\Include
DirText "This will install Test1 on your computer. Choose a folder:"
Section "InstallDirTest1"
    SetOutPath $INSTDIR
    File "Include\*.*"
SectionEnd
UninstallText "Uninstall"
UninstallExeName "test1_uninst.exe"
Section Uninstall
!Include "Delete1.pis"
SectionEnd
To generate test files:

Code: [Select]

#include <stdio.h>
#include <direct.h>
int __cdecl main(void)
{
FILE *f1, *f2;
char name[50];
_mkdir("Include");
f1 = fopen("Delete1.pis", "w");
for (int i=0; i<1000; i++) {
sprintf(name, "Include/file%d.h", i);
f2 = fopen(name, "w");
if (f2) fclose(f2);
int len = sprintf(name, "Delete Include\\file%d.h\n", i);
fwrite(name, len, 1, f1);
}
if (f1) fclose(f1);
return 0;
} poinst.exe crash with value over 915.

[frankie]

Pelle my disk is OK, but anyway the old V8.00 poinst works well with same installer script.
Seems a typical memory corruption problem. If the number of files before crash is not constant maybe the problem is in dynamic memory allocation of files array?