On Mon, 2012-11-05 at 08:21 +0000, Lee Jones wrote: > > > processed = sscanf(str, "@%lli:%u%n:%d%n", > > > - &base, &resources[1].start, &consumed, > > > + &base, (unsigned int *)&resources[1].start, &consumed, > > > &vm_cmdline_id, &consumed); > > > > This suppresses a valid warning, leaving our code no less wrong than > > before. But now it's silently wrong! > > > > Do you want to try again? > > For sure I'll try again. > > How does `s/%u/%p/` suit?
sscanf doesn't do "%p"... The only reasonable way of fixing this is a intermediate local variable - will post a patch in a second. Thanks for spotting this! Paweł -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
