View Issue Details

IDProjectCategoryView StatusLast Update
0004250unrealmodule apipublic2015-05-23 17:49
Reporters0beit Assigned Tosyzop  
PrioritynormalSeveritytweakReproducibilityalways
Status resolvedResolutionfixed 
Product Version3.2.10.2 
Fixed in Version3.4-alpha3 
Summary0004250: HOOKTYPE_RAWPACKET_IN parameters inconsistent with HOOKTYPE_PACKET
DescriptionUnlike HOOKTYPE_PACKET, I am unable to modify the length of the buffer passed through HOOKTYPE_RAWPACKET_IN.

This is because HOOKTYPE_PACKET passes int*, while HOOKTYPE_RAWPACKET_IN passes int (non-pointer value), this means you are unable to modify the length of a buffer if it changes going in the opposite direction.
Additional InformationIt is perhaps possible to return 0 (so that the function stops executing) and emulate the rest of read_packet within the callback, this is a hacky workaround but it can be done in a pinch I suppose.
TagsNo tags attached.
3rd party modules

Activities

syzop

2014-01-12 10:26

administrator   ~0017872

I'm pretty sure this was intentional because changing the length (at that point) can potentially cause issues.

s0beit

2014-01-13 11:58

reporter   ~0017891

For a module I'm making it is required that I modify the data in and out (and it must be the raw packet)

Not sure what issues you're talking about specifically.

syzop

2015-05-23 17:49

administrator   ~0018332

Changed in 3.4-alpha3:
https://github.com/unrealircd/unrealircd/commit/c66d21374821e6990c42490cc9ce479fe3573bdf
Module coders: HOOKTYPE_RAWPACKET_IN: third argument is now int * rather than int so you can change the length (and readbuf is of 8K size, so should be plenty). Requested by s0beit in 0004250.

Issue History

Date Modified Username Field Change
2013-12-15 08:04 s0beit New Issue
2014-01-12 10:25 syzop Priority high => normal
2014-01-12 10:26 syzop Note Added: 0017872
2014-01-13 11:58 s0beit Note Added: 0017891
2014-03-14 01:14 peterkingalexander Issue cloned: 0004261
2015-05-23 17:49 syzop Note Added: 0018332
2015-05-23 17:49 syzop Status new => resolved
2015-05-23 17:49 syzop Fixed in Version => 3.4-alpha3
2015-05-23 17:49 syzop Resolution open => fixed
2015-05-23 17:49 syzop Assigned To => syzop
2017-01-06 15:48 syzop Category module => module api