[vdr] vdr 1.3.44: more format string checking

Darren Salt linux at youmustbejoking.demon.co.uk
Wed Mar 15 19:27:59 CET 2006


I demand that C.Y.M may or may not have written...

> Darren Salt wrote:
>> I've done a build of vdr 1.3.44 with -Wformat=2. This was noisier than it
>> might be due to some tr() calls; however, in <libintl.h>, I've spotted
>> something of use:
>>   __attribute_format_arg__ (index)
[snip]

> With this patch applied, I get a lot of these (null) messages in my log:

> Mar 15 04:07:31 sid vdr: [26388] (null) thread started (pid=26388, tid=26388)
> Mar 15 04:07:31 sid vdr: [26389] (null) thread started (pid=26389, tid=26389)

Presumably, the cThread object is being initialised with a null description.
This is cosmetic, but the attached patch, applied on top of my previous
patch, should fix it.

-- 
| Darren Salt    | linux or ds at              | nr. Ashington, | Toon
| RISC OS, Linux | youmustbejoking,demon,co,uk | Northumberland | Army
| + Buy local produce. Try to walk or cycle. TRANSPORT CAUSES GLOBAL WARMING.

Reality is for people who can't handle computers.
-------------- next part --------------
--- vdr-1.3.44~/thread.c
+++ vdr-1.3.44/thread.c
@@ -208,7 +208,8 @@
   childTid = 0;
   childThreadId = 0;
   description = NULL;
-  SetDescription("%s", Description);
+  if (Description)
+    SetDescription("%s", Description);
 }
 
 cThread::~cThread()


More information about the vdr mailing list