summaryrefslogtreecommitdiff
path: root/lcdproc.c
diff options
context:
space:
mode:
authorJoachim Wilke <vdr@joachim-wilke.de>2011-01-01 16:58:38 +0100
committerJoachim Wilke <vdr@joachim-wilke.de>2011-01-09 18:12:22 +0100
commitefcb6216fad91fb8a71aa7aff4c8b3b055e2cb69 (patch)
tree063d62ed737d3a69b1a4274ac03ae520afdaa123 /lcdproc.c
parentc19273f938a73cb6507d10eef6e83955e76c9a00 (diff)
downloadvdr-plugin-lcdproc-efcb6216fad91fb8a71aa7aff4c8b3b055e2cb69.tar.gz
vdr-plugin-lcdproc-efcb6216fad91fb8a71aa7aff4c8b3b055e2cb69.tar.bz2
Fix compiler warnings
- Added several asprintfs result checks - Fixed a sprintf call
Diffstat (limited to 'lcdproc.c')
-rw-r--r--lcdproc.c20
1 files changed, 16 insertions, 4 deletions
diff --git a/lcdproc.c b/lcdproc.c
index 4ce0385..ff55b99 100644
--- a/lcdproc.c
+++ b/lcdproc.c
@@ -121,7 +121,10 @@ void cLcdFeed::SetAudioTrack(int Index, const char * const *Tracks)
OsdTitle(trVDR("Button$Audio"));
if (AudioTrack)
free(AudioTrack);
- asprintf(&AudioTrack, "%s", Tracks[Index]);
+ if (asprintf(&AudioTrack, "%s", Tracks[Index]) < 0) {
+ syslog(LOG_ERR, "lcdproc: error allocating memory in asprintf");
+ return;
+ }
OsdCurrentItem(AudioTrack);
}
@@ -129,13 +132,22 @@ void cLcdFeed::SetAudioChannel(int AudioChannel){
char * TrackDescription;
switch (AudioChannel){
case 0:
- asprintf(&TrackDescription, "%s (%s)", AudioTrack, tr("Stereo"));
+ if (asprintf(&TrackDescription, "%s (%s)", AudioTrack, tr("Stereo")) < 0) {
+ syslog(LOG_ERR, "lcdproc: error allocating memory in asprintf");
+ return;
+ }
break;
case 1:
- asprintf(&TrackDescription, "%s (%s)", AudioTrack, tr("Left channel"));
+ if (asprintf(&TrackDescription, "%s (%s)", AudioTrack, tr("Left channel")) < 0) {
+ syslog(LOG_ERR, "lcdproc: error allocating memory in asprintf");
+ return;
+ }
break;
case 2:
- asprintf(&TrackDescription, "%s (%s)", AudioTrack, tr("Right channel"));
+ if (asprintf(&TrackDescription, "%s (%s)", AudioTrack, tr("Right channel")) < 0) {
+ syslog(LOG_ERR, "lcdproc: error allocating memory in asprintf");
+ return;
+ }
break;
default:
return;