Typo in error msg when no crashkernel memory reservation is set

Bug #1670696 reported by Eric Desrochers
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
makedumpfile (Debian)
Fix Released
Unknown
makedumpfile (Ubuntu)
Fix Released
Medium
Unassigned
Nominated for Bionic by Thadeu Lima de Souza Cascardo
Xenial
Won't Fix
Medium
Unassigned
Zesty
Won't Fix
Medium
Unassigned

Bug Description

[Impact]

 * There is a typo in ERRMSG of function show_mem_usage affecting Xenial/Zesty, Debian and makedumpfile upstream.

This should be :
"show_mem_usage: No memory is reserved for crashkernel!"

instead of :
"show_mem_usage: No memory is reserved for crashkenrel!"

The typo is located here in src code :

# filename : makedumpfile.c
-
        int show_mem_usage(void)
         {
         uint64_t vmcoreinfo_addr, vmcoreinfo_len;
         struct cycle cycle = {0};

         if (!is_crashkernel_mem_reserved()) {
               ==>  ERRMSG("No memory is reserved for crashkenrel!\n");
                 return FALSE;
         }
-

[Test Case]

 * Take a system with no crashkernel memory reservation set
 * run command : makedumpfile --mem-usage /proc/kcore

 * Error message :
show_mem_usage: No memory is reserved for crashkenrel!

[Regression Potential]

 * No regression, it is a trivial change with no behavioural change.

[Other Info]

* Upstream:
Patch has been submitted upstream (ML :<email address hidden>)
- AFAIK, there is no upstream bug database, so no URL available, ML only.
- Patch will be merge in upstream version 1.6.2 # See comment #4

* Debian bug :
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=857051

Eric Desrochers (slashd)
Changed in makedumpfile (Ubuntu):
importance: Undecided → Low
assignee: nobody → Eric Desrochers (slashd)
description: updated
Eric Desrochers (slashd)
description: updated
summary: - Error msg typo when no crashkernel memory reservation
+ Typo in error msg when no crashkernel memory reservation is set
Eric Desrochers (slashd)
description: updated
Revision history for this message
Eric Desrochers (slashd) wrote :

Patch has been submitted to Debian

Eric Desrochers (slashd)
Changed in makedumpfile (Ubuntu Xenial):
importance: Undecided → Low
assignee: nobody → Eric Desrochers (slashd)
Changed in makedumpfile (Ubuntu Zesty):
status: New → Confirmed
Changed in makedumpfile (Ubuntu Xenial):
status: New → Confirmed
status: Confirmed → In Progress
Changed in makedumpfile (Ubuntu Zesty):
status: Confirmed → In Progress
Revision history for this message
Eric Desrochers (slashd) wrote :

Xenial debdiff

description: updated
Eric Desrochers (slashd)
description: updated
Eric Desrochers (slashd)
description: updated
description: updated
Changed in makedumpfile (Ubuntu Xenial):
importance: Low → Medium
Changed in makedumpfile (Ubuntu Zesty):
importance: Low → Medium
Revision history for this message
Eric Desrochers (slashd) wrote :

Message from makedumpfile maintainer :

"
Hello Eric,

Thanks, I'll merge this fix into v1.6.2.

P.S. I fixed other typos on this occasion.

Regards,
Atsushi Kumagai
"

- Eric

Eric Desrochers (slashd)
description: updated
Revision history for this message
Eric Desrochers (slashd) wrote :

Zesty debdiff

Eric Desrochers (slashd)
description: updated
Revision history for this message
Eric Desrochers (slashd) wrote :
Revision history for this message
Brian Murray (brian-murray) wrote :

For the record, I don't think this is worth SRU'ing to a stable release unless it is bundled with other non-typo fixes.

Eric Desrochers (slashd)
Changed in makedumpfile (Ubuntu Xenial):
status: In Progress → Won't Fix
Changed in makedumpfile (Debian):
status: Unknown → New
Eric Desrochers (slashd)
tags: added: sts-sru
tags: removed: sts-sru
Eric Desrochers (slashd)
Changed in makedumpfile (Ubuntu):
assignee: Eric Desrochers (slashd) → nobody
Changed in makedumpfile (Ubuntu Xenial):
assignee: Eric Desrochers (slashd) → nobody
Changed in makedumpfile (Ubuntu Zesty):
assignee: Eric Desrochers (slashd) → nobody
Eric Desrochers (slashd)
Changed in makedumpfile (Ubuntu Zesty):
status: In Progress → Won't Fix
Revision history for this message
Eric Desrochers (slashd) wrote :

For the record, the change has been merge in makedumpfile upstream:

--
commit ade8512bd2be9eef9f2a0376b41f22d5417c72f3
Author: Eric Desrochers <email address hidden>
Date: Wed Mar 8 09:45:02 2017 +0900

    [PATCH] Fix typo in the ERRMSG of function show_mem_usage

    Should be "kernel" instead of "kenrel".

    Signed-off-by: Eric Desrochers <email address hidden>
--

Changed in makedumpfile (Debian):
status: New → Fix Released
Changed in makedumpfile (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.