search

mate-desktop / mate-power-manager

Power management tool for the MATE desktop
https://mate-desktop.org
GNU General Public License v2.0
59 stars 51 forks source link

Remove warning use of memory after it is freed #329

Closed rbuj closed 4 years ago

raveit65 commented 4 years ago

Confirmed, that PR fixes both warnings.

 CC       egg-idletime.o

egg-idletime.c:334:2: warning: Use of memory after it is freed

        g_ptr_array_remove (idletime->priv->array, alarm);

        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

egg-idletime.c:446:42: warning: This statement is never executed

        for (i=0; i<idletime->priv->array->len; i++) {

                                                ^

2 warnings generated.

Build log looks good at this point. https://travis-ci.org/mate-desktop/mate-power-manager/jobs/659348088#L2471