[Bro-Dev] #267: Patch to fix internal log_encryption_key handling

[Bro Tracker]

#267: Patch to fix internal log_encryption_key handling
---------------------+--------------------
  Reporter:  seth    |      Owner:
      Type:  Patch   |     Status:  new
  Priority:  Normal  |  Milestone:  Bro1.6
 Component:  Bro     |    Version:  1.5.2
Resolution:          |   Keywords:  sprint
---------------------+--------------------

Comment (by seth):

 Replying to [comment:2 robin]:
 > What was the problem here? The original way of accessing the variable
 should work.

 Not sure.  It was giving me this error..

 {{{
 Program received signal EXC_BAD_ACCESS, Could not access memory.
 Reason: KERN_INVALID_ADDRESS at address: 0x0000000000000030
 0x00000001000958a9 in Val::AsString (this=0x0) at Val.h:241
 241             CONST_ACCESSOR(TYPE_STRING, BroString*, string_val,
 AsString)
 (gdb) bt
 #0  0x00000001000958a9 in Val::AsString (this=0x0) at Val.h:241
 #1  0x000000010012f5db in BroFile::SetAttrs (this=0x100bd02b0,
 arg_attrs=0x100bd0140) at /Users/seth/bro/bro.git/src/File.cc:503
 #2  0x00000001001666fb in ID::UpdateValAttrs (this=0x100bcfb90) at
 /Users/seth/bro/bro.git/src/ID.cc:221
 #3  0x0000000100235b2b in make_var (id=0x100bcfb90, t=0x100986060,
 c=INIT_FULL, init=0x100bcffd0, attr=0x100bd00d0, dt=VAR_REGULAR,
 do_init=1) at /Users/seth/bro/bro.git/src/Var.cc:172
 #4  0x0000000100235eaf in add_global (id=0x100bcfb90, t=0x0, c=INIT_FULL,
 init=0x100bcffd0, attr=0x100bd00d0, dt=VAR_REGULAR) at
 /Users/seth/bro/bro.git/src/Var.cc:179
 #5  0x000000010008331f in yyparse () at parse.y:800
 #6  0x000000010009365a in main (argc=2, argv=0x7fff5fbfeee8) at
 /Users/seth/bro/bro.git/src/main.cc:741
 (gdb) up
 #1  0x000000010012f5db in BroFile::SetAttrs (this=0x100bd02b0,
 arg_attrs=0x100bd0140) at /Users/seth/bro/bro.git/src/File.cc:503
 503
 InitEncrypt(log_encryption_key->AsString()->CheckString());
 (gdb) down
 #0  0x00000001000958a9 in Val::AsString (this=0x0) at Val.h:241
 241             CONST_ACCESSOR(TYPE_STRING, BroString*, string_val,
 AsString)
 (gdb) print this
 $1 = (const Val * const) 0x0
 }}}

-- 
Ticket URL: <http://tracker.icir.org/bro/ticket/267#comment:4>
Bro Tracker <http://tracker.icir.org/bro>
Bro Issue Tracker