On 29/08/2014 17:15, Stephan Aßmus wrote: > Am 29.08.2014 17:04, schrieb François Revol: >> On 29/08/2014 17:00, François Revol wrote: >>> On 29/08/2014 16:50, Jérôme Duval wrote: >>>> 2014-08-29 12:32 GMT+02:00 <revol@xxxxxxx>: >>>>> + device = new(std::nothrow) >>>>> SerialDevice(&sSupportedDevices[0], >>>>> + sHardcodedPorts[i].ioBase, >>>>> sHardcodedPorts[i].irq); >>>>> + if (device && device->Probe()) >>>> >>>> You should check against NULL. >> >> Actually it is here, just not obvious. > > He means the style guide says you need to write this: > > if (device != NULL && device->Probe()) Which I just did :p François.