Skip to content
  • Steven Rostedt (Red Hat)'s avatar
    tracing: Put back comma for empty fields in boot string parsing · 43ed3843
    Steven Rostedt (Red Hat) authored
    
    
    Both early_enable_events() and apply_trace_boot_options() parse a boot
    string that may get parsed later on. They both use strsep() which converts a
    comma into a nul character. To still allow the boot string to be parsed
    again the same way, the nul character gets converted back to a comma after
    the token is processed.
    
    The problem is that these two functions check for an empty parameter (two
    commas in a row ",,"), and continue the loop if the parameter is empty, but
    fails to place the comma back. In this case, the second parsing will end at
    this blank field, and not process fields afterward.
    
    In most cases, users should not have an empty field, but if its going to be
    checked, the code might as well be correct.
    
    Signed-off-by: default avatarSteven Rostedt <rostedt@goodmis.org>
    43ed3843