Re: Re: quit files
| Newsgroups | gmane.comp.emulators.hercules390.general |
|---|---|
| Message-ID | <CAPcd4G_FLJcc_vUOjhJncT4_SzQzg6rEoh=6tuPWXxq-VLNgZQ@mail.gmail.com> |
So i've traced the release_config code in config.c (because debugs show the
code entering HDL: release config) as far as detach_devblk in config.c down
to this...
/* Free the argv array */
for (i = 0; i < dev->argc; i++)
if (dev->argv[i])
free(dev->argv[i]);
if (dev->argv)
free(dev->argv);
How can you have an array item (argv[i]) referenced by a non-array
statement (argv)? Plus, when the loop falls thru because i = argc, then
what happens? Plus, dont you need braces on the for loop like this?
/* Free the argv array */
for (i = 0; i < dev->argc; i++)
{
if (dev->argv[i])
free(dev->argv[i]);
}
if (dev->argv)
free(dev->argv);
Joe
On Sun, Jun 23, 2019 at 6:29 AM [email protected] [hercules-390] <
[email protected]> wrote:
>
>
> Thanks Joe, yes, I know. That‘s the particular reason why I think it‘s
> that other bug I‘m chasing. Cheers, Jürgen
>
>