Opened 14 years ago

Closed 10 years ago

#26675 closed enhancement (wontfix)

wireshark 1.4.0: add variant +setuid_root to install dumpcap as setuid root

Reported by: mklein-de (Michael Klein) Owned by: ghosthound
Priority: Normal Milestone:
Component: ports Version: 1.9.1
Keywords: haspatch Cc: opendarwin.org@…
Port: wireshark

Description

To allow a ordinary users to start capturing packets in wireshark right away, it would be nice if the dumpcap binary could be installed as setuid root. This feature is already present in wireshark's configure script (--enable-setuid-install).

Attached is a patch that adds a +setuid_root variant to the Portfile.

Regards, Michael

Attachments (1)

wireshark+setuid_root.diff (395 bytes) - added by mklein-de (Michael Klein) 14 years ago.
Portfile patch

Download all attachments as: .zip

Change History (4)

Changed 14 years ago by mklein-de (Michael Klein)

Attachment: wireshark+setuid_root.diff added

Portfile patch

comment:1 Changed 14 years ago by ghosthound

Owner: changed from macports-tickets@… to ricci@…
Status: newassigned

The solution in #26293 is preferable (make /dev/bpf* readable by members of the admin group), is there a reason that won't work for you?

comment:2 Changed 14 years ago by mklein-de (Michael Klein)

I guess #26293 would also work for me (haven't seen it before).

May I ask why you think #26293 is preferable? IMHO both solutions have their own pros and cons.

Michael

comment:3 Changed 10 years ago by ghosthound

Resolution: wontfix
Status: assignedclosed

setuid is a bad idea (tm).

Note: See TracTickets for help on using tickets.