Opened 4 years ago

Last modified 5 months ago

#21119 new enhancement

Deprecate and replace type_getarray_val()

Reported by: nickm Owned by:
Priority: Medium Milestone:
Component: Core Tor/Trunnel Version:
Severity: Normal Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


In the first version of the introduce1 branch, the type_getarray_val() accessor was used erroneously in a way that could have resulted in a buffer overflow.

That bodes ill for its usability. And since Trunnel is supposed to make it hard for us to make mistakes, we should consider fixing this.

Brainstorming replacement ideas:

  • type_copyinto_val(object, input, input_len);
  • type_copyfrom_val(object, target, target_len);
  • type_getarray_checked(object, array, need_len);

Child Tickets

Change History (2)

comment:1 Changed 14 months ago by nickm

Owner: nickm deleted
Status: newassigned

These tickets are not things I'm currently working on. They may be important, but they don't need to be done by me specifically. Un-assigning.

comment:2 Changed 5 months ago by teor

Status: assignednew

Change tickets that are assigned to nobody to "new".

Note: See TracTickets for help on using tickets.