aboutsummaryrefslogtreecommitdiff
path: root/cddl/contrib/opensolaris/common/ctf
Commit message (Collapse)AuthorAgeFilesLines
* libctf: Don't construct pointers to out of bounds array offsetsConrad Meyer2018-03-261-3/+5
| | | | | | | | | | | | Just attempting to do the pointer arithmetic is undefined behavior. No functional change intended. Reported by: Coverity Sponsored by: Dell EMC Isilon Notes: svn path=/head/; revision=331595
* Recursively enumerate anonymous structs and unions in ctf_member_info().Mark Johnston2016-08-301-7/+24
| | | | | | | | | | | | Previously, ctf_member_info() would ignore members belonging to an anonymous struct or union. This made it impossible to, for example, trace the m_next field of an mbuf using DTrace. Reported and tested by: gallatin MFC after: 2 weeks Notes: svn path=/head/; revision=305055
* ctf_add_type(): when looking up an integer or floating point type in theMark Johnston2015-05-101-4/+17
| | | | | | | | | | | | | | | | | | list of pending dynamic type definitions, a match on the type name is not sufficient - we need to compare the type encodings as well. For example, bitfields have their own distinct type definitions which share the name of the underlying integer type, and these types aren't generally interchangeable. This bug was causing the following libdtrace error when attempting to trace the th_flags member of a struct tcphdr: cg: bad field: off 104 type <32877> bits 539620016 Reported by: rwatson MFC after: 3 weeks Notes: svn path=/head/; revision=282739
* When copying a type from a source CTF container to a destination container,Mark Johnston2015-03-111-4/+7
| | | | | | | | | | | | | | | | | | | | | ctf_add_type() first performs a by-name lookup of the type in the destination container. If this lookup returns a forward declaration for an enum, struct, or union, reset dst_type back to CTF_ERR, indicating that the source type is not in fact present in the destination container. This ensures that ctf_add_type() will also search the destination container's dynamic type list for the source type. Without this change, a pair of mutually recursive struct definitions could cause infinite recursion in ctf_add_type() if the destination container only contained forward declarations for the struct types: ctf_add_type() recursively calls itself on each struct member's type, and the forward declarations meant that the dynamic type list search would be skipped. MFC after: 2 weeks Sponsored by: EMC / Isilon Storage Division Notes: svn path=/head/; revision=279869
* CTF containers use the ctf_dtoldid field as a threshold type index whichMark Johnston2015-03-101-2/+2
| | | | | | | | | | | | | | | | | | | | indicates the range of type indices which have been committed to the container by ctf_update(). However, the top bit of the dtd_type field is not part of the type index; rather, it is a flag used to indicate that the corresponding CTF container is a parent. This is why the maximum CTF type index is 2^15 - 1 rather than 2^16 - 1. Therefore, this flag must be masked off (using the CTF_TYPE_TO_INDEX macro) when comparing a type index with the ctf_dtoldid field of a container. This bug was causing libctf to erroneously free committed type definitions in ctf_discard(). libdtrace holds some references to such types, resulting in a use-after-free. MFC after: 2 weeks Sponsored by: EMC / Isilon Storage Division Notes: svn path=/head/; revision=279864
* ctf_discard(): fetch the next list element before restarting the loop. IfMark Johnston2015-03-101-1/+1
| | | | | | | | | | | | | we end up skipping a dynamic type because it has already been committed to the container, we would previously either set the loop variable to an uninitialized local variable, or set it to itself, resulting in an infinite loop. MFC after: 2 weeks Sponsored by: EMC / Isilon Storage Division Notes: svn path=/head/; revision=279862
* MFV illumosRui Paulo2014-06-262-9/+120
| | | | | | | | | | | | | 4474 DTrace Userland CTF Support 4475 DTrace userland Keyword 4476 DTrace tests should be better citizens 4479 pid provider types 4480 dof emulation is missing checks MFC after: 2 weeks Notes: svn path=/head/; revision=267941
* Quiesce warning assigning to void * from const ctf_header_t * by explicity ↵Sean Bruno2013-11-041-1/+1
| | | | | | | | | | | casting to void * before assignment. Submitted as Illumos issue 4287 Notes: svn path=/head/; revision=257657
* MFV r254422:Xin LI2013-08-234-16/+204
| | | | | | | | | | | Illumos DTrace issues: 3089 want ::typedef 3094 libctf should support removing a dynamic type 3095 libctf does not validate arrays correctly 3096 libctf does not validate function types correctly Notes: svn path=/head/; revision=254744
* This commit was generated by cvs2svn to compensate for changes in r178525,John Birrell2008-04-2510-0/+4578
which included commits to RCS files with non-trunk default branches. Notes: svn path=/head/; revision=178526