missing check in src/mcconfig/get.c mc_config_get_groups()

[vit_r]

This obvious correction is raising some questions and may be other corrections. While experts can have a look i'll try
to prepare enveloping patch(to add to this ticket).

[andrew_b]

You cannot write access to the repo, so don't take ownership and accept of the ticket to youself.

[vit_r]

enveloping previous patch for discussing

[vit_r]

Its for discussing:

Seems, master (cloned 20091203-1103) has next five calls to this func:

I.
src/setup.c

gsize groups_count; <- _NOT_ used

504:

curr_grp = groups = mc_config_get_groups (tmp_cfg, &groups_count);
if (!groups)
{

mc_config_deinit(tmp_cfg);
return;

}

II.
src/filehighlight/ini-file-read.c

gsize ftype_names_size; <- _NOT_ used

251:

orig_group_names = group_names = mc_config_get_groups (fhl->config, &ftype_names_size);

if (group_names == NULL)

return FALSE;

III.
src/skin/colors.c

gsize items_count; <- _NOT_ used

259:

orig_groups = groups = mc_config_get_groups (mc_skin->config, &items_count);

if (groups == NULL)

return;

IV.
src/skin/colors.c

gsize items_count; <- is used, - see L 307

294:

orig_groups = groups = mc_config_get_groups (mc_skin->config, &items_count);

if (groups == NULL

*groups == NULL) {

if (groups != NULL)

g_strfreev (groups);

return FALSE;

}

307:

orig_keys = keys = mc_config_get_keys (mc_skin->config, *groups, &items_count);

if (keys == NULL

*keys == NULL) {

if (keys != NULL)

g_strfreev (keys);

continue;

}

V.
src/mcconfig/common.c

210:

groups = mc_config_get_groups (tmp_config, NULL);

for (curr_grp = groups; *curr_grp != NULL; curr_grp++) {

--

In cases I, II and III _NOT_ used vars are deleted

In cases IV items_count is used _BUT_

if (groups == NULL ...) {

if (groups != NULL)

}

if (keys == NULL ...) {

if (keys != NULL)

}

look strange and may be want to be rewritten

In cases V added

if (!groups)

return FALSE;

In func added

return NULL;

But such questions:

1. Was there a chance for mc_config_get_groups () to return NULL ?
2. And seems src/mcconfig/*.c are out yet, because no src/mcconfig/*.o after make appear then why they are there? New stuff? Hope above will be not in vain

[vit_r]

Replying to andrew_b:

You cannot write access to the repo, so don't take ownership and accept of the ticket to youself.

Sorry
i know, but when ticket was created it did not apeare in My_tickets-list (more then one hour). And it was assigned as my and searching by number was found.
And I thought the Trac should prevent my wrong action.
Can You explain what to do in such case, please.
And reassign it as new one instead of accepted , please.
I don't know why this happened. Next time I will be more careful

vit_r

[vit_r]

Doubled fix

[slavazanko]

Will applied into 1845