After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 342820 - gstnetclientclock.c(453): servaddr used before set
gstnetclientclock.c(453): servaddr used before set
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
0.10.x
Other Linux
: Normal normal
: 0.10.7
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-05-24 16:37 UTC by Stanislav Brabec
Modified: 2006-05-25 15:52 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Stanislav Brabec 2006-05-24 16:37:24 UTC
We got a following bug report from David Binderman:

I just tried to compile package gstreamer010-0.10.4-11 with the Intel C
compiler.

It said

gstnetclientclock.c(453): warning #592: variable "servaddr" is used before its
value is set

The source code is

  GST_DEBUG_OBJECT (self, "socket opened on UDP port %hd",
      ntohs (servaddr.sin_port));

I agree with the compiler. Suggest initialise local variable "servaddr" 
before first use.

I do not see exact solution, but it can be:
- servaddr instead of self->servaddr
- bad order of commands
Comment 1 Wim Taymans 2006-05-25 15:52:38 UTC
        * libs/gst/net/gstnetclientclock.c: (gst_net_client_clock_start):
        Initialize variable before using. Fixes #342820.