Bug 114281 - Two compiler warnings in gnome-pilot
Summary: Two compiler warnings in gnome-pilot
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: gnome-pilot
Version: 1
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Dave Malcolm
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2004-01-26 10:28 UTC by d.binderman
Modified: 2007-11-30 22:10 UTC (History)
0 users

Fixed In Version: 2.0.12-3
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2004-09-21 17:11:00 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description d.binderman 2004-01-26 10:28:43 UTC
Description of problem:

I just tried to compile package gnome-pilot-2.0.10-4, from Redhat
Fedora Core 1.

The compiler said

1.

pilot.c(401): warning #1011: missing return statement at end of
non-void function "free_conduit_list"

The source code is

void *
free_conduit_list (GList *conduits)
{
        /* FIXME Properly free each state */

        g_list_free (conduits);
}

Looks like a missing return statement to me.

2.

gpilotd-client.c(420): warning #140: too many arguments in function call

The source code is

	get_system_info (gpc);

but, earlier in the file

static void
get_system_info ()
{

Suggest remove gpc from call to get_system_info.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Dave Malcolm 2004-09-21 17:11:00 UTC
re (1):  This is still present in gnome-pilot-2.0.12;
capplet/pilot.[ch]: free_conduit_list: looks like a bad argument;
should return void rather than void*    It appears that it never gets
called, so it looks like a memory leak, rather than a crash.

re (2): This is also still present in gnome-pilot-2.0.12;
utils/gpilotd-client.c: your suggestion of removing the gpc from the
call appears to be the correct fix (should also have an explicit void
in the declaration of the function)

I've fixed these and rebuilt into rawhide as gnome-pilot-2.0.12-3

I've reported the patch upstream here:
http://bugzilla.gnome.org/show_bug.cgi?id=153317


Note You need to log in before you can comment on or make changes to this bug.