Menu

#1371 Fixed nsDebug token settings

backport-needed
open
nobody
None
5
2018-11-20
2018-06-19
No
  • removed faulty comment
  • set correct table name
  • including all debug tokens
  • setting correct off value to 2 instead of 0

This solves https://sourceforge.net/p/net-snmp/bugs/2874/

1 Attachments

Discussion

  • Bart Van Assche

    Bart Van Assche - 2018-06-22

    The last part of this patch looks suspicious to me. Are you sure the following changes are correct?

                 if (!debug_entry)
                     continue;
    -           status = (debug_entry->enabled ? RS_ACTIVE : RS_NOTINSERVICE);
    +           status = debug_entry->enabled;
                snmp_set_var_typed_value(request->requestvb, ASN_INTEGER,
                                          (u_char*)&status, sizeof(status));
    
                     if (debug_entry)
    -                    debug_entry->enabled =
    -                        (*request->requestvb->val.integer == RS_ACTIVE);
    +                   debug_entry->enabled = *request->requestvb->val.integer;
                    break
    
     
  • Anders Wallin

    Anders Wallin - 2018-06-25

    It' sort of correct, BUT appended is a much simplier patch fixing the problem at the root.

     
  • Anders Wallin

    Anders Wallin - 2018-11-19

    Is the last version of the patch OK?

     
  • Bart Van Assche

    Bart Van Assche - 2018-11-20

    Thanks for the patch. It has been applied on the v5.8 and master branches.

     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.