Permalink
Browse files

Revert "Initialise all the pointers in GetNvidiaRates()'s buf[256] to

NULL. Coverity 746855"

This is causing a segfault at frontend startup for me, switching it to
use memset() instead causes the same problem. I suspect that 256 is
far larger than it needs to be but I'm not sure whether that's causing
us to run out of memory?

This reverts commit f1d33c5.
  • Loading branch information...
stuartm committed May 27, 2013
1 parent 967230b commit fb4106477c9a8f0860a2df00226bda07f9c24422
Showing with 1 addition and 1 deletion.
  1. +1 −1 mythtv/libs/libmythui/util-nvctrl.cpp
@@ -274,7 +274,7 @@ int GetNvidiaRates(t_screenrate& screenmap)
{
int w, h, vfl, hfl, i, irate;
double dcl, r;
char *buf[256] = {NULL};
char *buf[256];
uint64_t key, key2;
// skip name

0 comments on commit fb41064

Please sign in to comment.