diff options
author | Joachim Wilke <vdr@joachim-wilke.de> | 2011-01-01 16:58:38 +0100 |
---|---|---|
committer | Joachim Wilke <vdr@joachim-wilke.de> | 2011-01-09 18:12:22 +0100 |
commit | efcb6216fad91fb8a71aa7aff4c8b3b055e2cb69 (patch) | |
tree | 063d62ed737d3a69b1a4274ac03ae520afdaa123 /lcdproc.c | |
parent | c19273f938a73cb6507d10eef6e83955e76c9a00 (diff) | |
download | vdr-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.c | 20 |
1 files changed, 16 insertions, 4 deletions
@@ -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; |