On Tuesday, February 3, 2015 8:03 AM, Dave Anderson <anderson@xxxxxxxxxx> wrote:
> Derek,
> I don't understand what you're expecting me to do -- pick one of the 4 patches?
When I repost, only the last one is the updated patch,
> Can you please repost just one patch?
> And this part clearly cannot be applied, because the "SysRq :" string would
> also be displayed by all of the possible sysrq letters:
> + while (!msg_found && fgets(buf, BUFSIZE, pc->tmpfile)) {
> + if (strstr(buf, "SysRq : "))
> + msg_found = TRUE;
> + }
So, are you ok with other lines of the patch?
I've just found the code has these lines below, will update the patch and repost,
while (!msg_found && fgets(buf, BUFSIZE, pc->tmpfile)) {
if (strstr(buf, "SysRq : Netdump") ||
strstr(buf, "SysRq : Trigger a crashdump") ||
strstr(buf, "SysRq : Crash") ||
strstr(buf, "SysRq : Trigger a crash")) {
pc->flags |= SYSRQ;
msg_found = TRUE;
}
}
Thanks,
--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility
