[asterisk-dev] [Code Review] remove unneeded explicit specification of channel in dahdi ioctls

Tzafrir Cohen tzafrir.cohen at xorcom.com
Wed Mar 24 14:14:56 CDT 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/584/
-----------------------------------------------------------

Review request for Asterisk Developers.


Summary
-------

Several DAHDI ioctls allow passing the channel numbers explicitly through one of the fields in the struct that is passed in the system call. This will override the actual file descriptor used. In some cases this is really needed. But in most cases it merely opens the door to unexpected bugs.

This patch removes a few such calls in Asterisk while not actually changing the behavior. The gain setting functions passed around a channel which is always 0, and thus this parameter is simply dropped.


Diffs
-----

  /trunk/channels/chan_dahdi.c 254360 

Diff: https://reviewboard.asterisk.org/r/584/diff


Testing
-------

So far: minimal: Does not seem to break anything here.


Thanks,

Tzafrir




More information about the asterisk-dev mailing list