io_submit.stp: fix array membership test

io_submit.stp keeps track of threads which are currently executing
in the io_submit system call like so:

probe syscall.io_submit {
          in_iosubmit[tid()] = 1
}

probe syscall.io_submit.return {
          /* this assumes a given proc will do lots of io_submit calls, and
           * so doesn't do the more expensive "delete in_iosubmit[p]".  If
           * there are lots of procs doing small number of io_submit calls,
           * the hash may grow pretty big, so using delete may be better
           */
          in_iosubmit[tid()] = 0
}

However, the test to see if a thread is currently executing in io_submit
is performed using the membership operator 'in':

  if (tid() in in_iosubmit)

This is obviously wrong.  We can do one of two things:
1) change the test to if (in_iosubmit[tid()] == 1) or
2) just perform the delete in the return probe

While I agree that we typically have a small number of threads performing
io_submit, I don't believe there is substance to the performance claims
for the delete operator.  So, I've opted for solution 2.

Signed-off-by: Jeff Moyer <jmoyer@redhat.com>

Updated spec file to change dependency on net-tools to iproute

rhbz1496156: removed dependencies on deprecated net-tools package

* java/stapbm.in: replaced netstat with ss

Change the http docker backend to run systemtap in the container image.

* httpd/backends.cxx (docker_backend::generate_module): Switch back to
  running systemtap inside the container. Trying to use the container
  image as a sysroot didn't work well. Trying to run systemtap on the
  sysroot worked fine, but running gcc 8 (f28) against a centos 7
  sysroot's kernel source failed. Trying to run the centos 7 gcc from f28
  kept crashing. So, we're back to running systemtap in the container
  image.

rpm: by default, configure with --enable-docs=prebuilt

Fix testcases broken by not expectig 'WARNING: Missing unwind data'

* lib/stap_run.exp: Fix systemtap.base/caller.exp
* systemtap.base/backtrace.exp: Fix itself
* systemtap.context/backtrace.tcl: Fix systemtap.context/context.exp

Fix testcase systemtap.string/text_str.exp

PR22991 (Invalid truncation of quoted strings) was fixed in
commit fa4f499d8c92600702853d763ae3ea45002c07e3, but respective
change hasn't been applied to systemtap.string/text_str.exp.
Adding the missing bit.

Fix testcase systemtap.unprivileged/unprivileged_embedded_C.exp

Commit 2f6fcfc6 (Fix BZ1546179 by being careful when accessing
sk_buff structures) made the "unprivileged_embedded_C.exp"
testcase unhappy.  It started to report false positives.

Thing is that embeddedc.awk can't correctly parse two block
boundaries at a line, which is exactly what the commit introduces
by adding "} catch { }", or "try { ... } catch { }".  Here is the
failure as observed with git stap 0e051c5:

=======
FAIL: unprivileged embedded C: : --unprivileged: Unterminated block at: function _input_route_type:long (skb:long): 2 0
FAIL: unprivileged embedded C: : --privilege=stapusr: Unterminated block at EOF: 2 0
FAIL: unprivileged embedded C: : --privilege=stapsys: Unterminated block at: function _input_route_type:long (skb:long): 2 0
FAIL: unprivileged embedded C: : --privilege=stapsys: Unterminated block at EOF: 2 0
FAIL: unprivileged embedded C (transitive): : --privilege=stapusr: Unterminated block at: function _input_route_type:long (skb:long): 2 0
FAIL: unprivileged embedded C (transitive): : --privilege=stapsys: Unterminated block at: function _input_route_type:long (skb:long): 2 0
=======

This commit tries to address the issue by making embeddedc.awk
able to process two block boundaries at a line.

stapbpf: add foreach loop support

* bpf-translate.cxx (visit_foreach_loop): translates foreach loops.

* bpf-internal.h (BPF_FUNC_map_get_next_key): add id for map_get_next_key.

* bpfinterp.cxx (bpf_interpret): call map_get_next_key when function id equals
  BPF_FUNC_map_get_next_key. (map_get_next_key): wrapper for bpf_get_next_key
  that includes support for limiting number of iterations and iterating in
  ascending/descending order.

* bpf_tests/foreach.stp: tests these changes.

Send the target executable over to the http server.

* httpd/backends.cxx (docker_backend::generate_module): Add the date and
  time to the docker image name, to make them easier to remove. Copy any
  target files from the zip file into the docker image.
* client-http.cxx (http_client::get_rpmname): If an rpm doesn't contain
  the file we're searching for, just return the empty string.
  (http_client_backend::include_file_or_directory): Only include the file
  or directory in the command line arguments if the callers asks.
  (http_client_backend::package_request): Move code from
  http_client_backend::find_and_connect_to_server(). If get_rpmname()
  returns the empty string, add the file so that it will get transferred
  over to the server.
  (http_client_backend::find_and_connect_to_server): Move code to
  http_client_backend::package_request().
* client-http.h (http_client_backend::include_file_or_directory): Updated
  prototype.
* client-nss.cxx (nss_client_backend::include_file_or_directory): Only
  include the file or directory in the command line arguments if the
  callers asks.
* client-nss.h (nss_client_backend::include_file_or_directory): Updated
  prototype.
* csclient.h: Updated client_backend::include_file_or_directory()
  prototype.

stapbpf: ensure uprobe DWARF offsets are 64-bits wide.

stapbpf.cxx: change type of DWARF offset from unsigned long to unsigned long long.

Merge branch 'master' of ssh://sourceware.org/git/systemtap

stapbpf: replace uint64_t with unsigned long for 32-bit compatibility

mips registers.stp: support oabi for kernel registers

stapbpf: add uprobe support

* bpf-translate.cxx (translate_bpf_pass): translate uprobe_derived_probes.

* stapbpf.cxx (collect_uprobes, register_uprobes, unregister_uprobes): read uprobe
  information from the .bo file, register/unregister uprobes via debugfs.

* tapsets.cxx (sort_for_bpf): create section names for each uprobe.

* tapsets.h: add necessary declarations.

* bpf_tests/uprobes.stp: smoke test for stapbpf uprobe support.

* bpf.exp: when running bpf_tests/uprobes.stp, spawn a process that triggers the test's probes.

dwflpp::resolve_prologue_endings should use dwarf_haspc, not highpc.

A function can have multiple non-consecutive ranges (for example when
a function is split into a hot and cold part, that are put in separate
sections). resolve_prologue_endings would use the highpc attribute of
a function DIE to see if an address was "beyond" the function (and so
also beyond the prologue). That could cause unexpected errors when
the DIE didn't have a highpc, but only a lowpc and ranges attribute.
Use dwarf_haspc () instead to test whether an address is inside a
function.

See also https://sourceware.org/ml/systemtap/2018-q2/msg00029.html

RHBZ1552745: staprun to set /proc/sys/kernel/kptr_restrict=1 on linux >= v4.15

Kernels >= 4.15 obscure /sys/module/$MODULE/section/$SECTION pointers
even to root users, unless kptr_restrict is set to 1.  Since we need
this information unobscured for module section-address configuration
at module startup, set that flag, unless $STAP_BZ1552745_OVERRIDE
exists.

Update mysql and xulrunner testapps

PR22991: Store escape sequence pointer before early break

runtime/stp_string.c (_stp_text_str): Capture the escape sequence
pointer before a possible while loop break instead of after. Needed
for correct truncation of UTF-8 literals. Also rename variable 'len'
to 'esc_len' for better clarity.

runtime/stp_string.c: Remove unused variable.

Variable was mistakenly added in commit fa4f499d8c.

PR22991: Fix invalid truncation of quoted strings

* runtime/stp_string.c (_stp_text_str): If truncation will remove part
  of an escape sequence, then truncate the entire sequence instead.

Make kernel DWARF unwinder work with ksalr.

The .debug_frame loaded from disk is already relocated against the
expected load offset of the kernel, but the actual static (load)
address might be different (with kaslr). So adjust the startLoc
for that difference when reading any address from the unwind table.

Document remote server testing in testsuite/lib/http_server.exp.

Revert "PR22193: don't elide regex match '=~' operator since it's now side-effecting"

This reverts commit 81e03c8fea43b3547a945306218f12722f78eb36, because it broke
elision (e.g. in tapset/linux/syscalls.stpm, see _af_inet_info_u()).

loc2stap.c: Give proper error message for unhandled DW_OP_GNU_parameter_ref.

Change the http docker backend to use the docker image as a sysroot.

* httpd/backends.cxx (docker_backend::generate_module): Switch from
  running systemtap in the docker container to mounting the container and
  using it as a sysroot.
* httpd/docker/fedora.json: Update the docker container to run
  "symlinks".
* httpd/docker/centos.json: Ditto.

RHBZ1566422 - fix spurious Build-id mismatch when probing reinserted kernel module

Code for newer kernels did not clear the address of the notes-section
when a probed module was unloaded. This caused spurious Build-id mismatch
when the module was reinserted as new addresses are not computed for
dynamically loaded modules (see also: PR23068) and the Build-id check
was trying to read the notes section at the no-longer-valid old address.

* runtime/sym.c (_stp_module_notifier): clear addresses on
MODULE_STATE_GOING in newer kernels (>=3.10) too.
* runtime/transport/symbols.c (_stp_kmodule_update_address): fix logic
error and clear notes section addr when reloc=NULL (aka. 'all').

Fix httpd/docker/fedora_install_package.py to correctly handle file paths.

* httpd/docker/fedora_install_package.py (PkgSystem.build_id_is_valid):
  Handle 'UsrMove' feature by just comparing file basenames, instead of
  full paths.

linux runtime: Add support for new kernel unwind fallback.

In newer kernels dump_trace got replaced by a new unwind infrastructure.
Add a new autoconf-unwind-stack-trace.c to detect whether we can use it.
Extend the runtime/stack.c _stp_stack_print_fallback with a new pt_regs*
argument. Update all callers and add dbug_unwind output to show which
fallback unwinder we are selecting (or if we are just giving up).
Rename the struct unwind_state in unwind.c and unwind.h to uw_state
because the old name now conflicts with the one used in the kernel.

Add a http server test case.

* testsuite/lib/http_server.exp: New file.
* testsuite/systemtap.http_server/http_server.exp: New test.
* testsuite/config/unix.exp: Added http_server.exp.
* testsuite/lib/systemtap.exp: Include server_ns.exp.

Fix the http server local backend to run the correct stap version.

* httpd/backends.cxx (local_backend::generate_module): Make sure we're
  running the correct version of systemtap.

Redirect all 'clog' output from the http server to the log file.

* httpd/main.cxx (parse_cmdline): Also send 'clog' output to the log file.
* nsscommon.cxx (start_log): Added an option to redirect 'clog' output to
  the log file.
* nsscommon.h: Update start_log() prototype.

Fix a small bug in follow_link().

* util.cxx (follow_link): Correctly terminate strings after calling
  readlink().

Improve http_client::get_rpmname().

* client-http.cxx (http_client::get_rpmname): Clean up memory leaks (found
  by valgrind) and improve/simplify the rpm finding logic.

Fix error in last client-http.cxx commit.

* client-http.cxx (http_client_backend::unpack_response): Fix error
  introduced in commit 82191ce.

Clean up some http server memory leaks found by valgrind.

* httpd/api.cxx (resource::~result_info): New function.
  (build_info::~build_info): Be sure to erase the result from the results
  list.
  (result_infos_erase): New function.
* httpd/backends.cxx (backends_atexit_handler): New function.
  (get_backends): Run backends_atexit_handler() at exit.
* httpd/backends.h: Add virtual backend_base destructor.
* httpd/server.cxx (get_key_values): Call json_object_put() to free the
  created json object.

The http server code now cleans up environment variables.

* httpd/api.cxx (response build_collection_rh::POST): Clean up the
  environment variable list the client sends the server.
* httpd/main.cxx (main): Clean up the environment variables.

stapbpf: remove BPF map copy logic from interpreter

bpfinterp.cxx, bpfinterp.h, stapbpf.cxx: Remove map structs and copy logic.
Instead, perform lookups/updates/deletes on BPF maps directly via bpf().

RHBZ1558350: fix typo in linux/nfsd.stp

* linux/nfsd.stp (probe nfsd.proc2.create): fix typo.

RHBZ1563052: fix off-by-one error in loc2c-runtime.h

The new version of kderef_string (added along with the new eBPF
runtime) copies an additional character compared to the old version.

* runtime/linux/loc2c-runtime.h (_stp_deref_string_nofault): reduce len by 1 to leave room for NUL terminator.
* testsuite/systemtap.base/set_kernel.stp: testcase for this bug.

The http client code now asks the server to delete the build info.

* client-http.cxx (http_client_backend::delete_op): Send a delete request
  to the server.
  (http_client_backend::unpack_response): After downloading everything,
  call delete_op().
* httpd/api.cxx (individual_build_rh::DELETE): New function.

BPF translator: improve register interference checking

bpf_opt.cxx (allocate): Apply the addition checks added to merge()
in commit 883ecae33bb. (reg_alloc): change while loop to for loop
to better indicate how num_spills is updated.

Handle removing http server temporary directories on shutdown.

* httpd/api.cxx (build_info::~build_info): Removes temporary directory.
  (build_info::module_build): Save temporary directory name.
  (client_request_data::~client_request_data): New function.
* httpd/api.h (client_request_data): Add destructor declaration.
* httpd/docker/stap_build_docker_image.py: Removes temporary directory.

Fix the systemtap testsuite LD_LIBRARY_PATH.

The goal here is to ensure the right dyninst libraries are used.
Systemtap is not only packaged for base RHEL/CentOS, but also as
a part of SCL (red hat developer toolset).  In case of SCL we need
to ensure that SCL version of systemtap tests against SCL version
of dyninst and the same is true for base os versions of both
components.

Before this change devtoolset version of systemtap was running
its tests against base rhel version of dyninst, and if that was
missing, dyninst tests were crashing.

This is a testsuite problem only, unrelated to normal stap usage.

Drop Python 2 subpackage in future systems

remote: fixes for bpf runtime

remote.cxx (class direct, stapsh): Set file extension to .bo and leave args
empty for bpf runtime.

PR22193: don't elide regex match '=~' operator since it's now side-effecting

NB: If we detected /* pragma: tagged_dfa */ earlier and set
session.need_tagged_dfa earlier, we would be able to safely elide
regex matches in programs that don't use the matched() tapset.

* elaborate.cxx (void_statement_reducer::visit_regex_query): Assume '=~' is side-effecting.
* staptree.h (varuse_collecting_visitor::visit_regex_query): New function.
* staptree.cxx (varuse_collecting_visitor::visit_regex_query): Assume '=~' is side-effecting.

* testsuite/runok/regex_noelide.stp: Make regex slightly more interesting.

Fix the naming convention problem for python-setuptools.

There is no python2-setuptools.rpm for any of rhel{6,7}.

EPEL6 has python2-setuptools.rpm, but it merely is a dummy wrapper
for python-setuptools.rpm.  ATM, there is no other reason to use EPEL
on RHEL than this, so it's better to avoid it entirely.  RHEL7
depsolver can pair python-setuptools.rpm with python2-setuptools.rpm
via Provides, but there is no python2-setuptools.rpm for RHEL7
either.  But Fedora does have python2-setuptools.rpm.

sysroot: fix short release -r option handling follow up

In case of sysroot set and short release -r option it does not make sense
to pass short release name into dwfl_linux_kernel_report_offline
function. This function is not aware about sysroot and it won't look for
kernel modules under sysroot directory.

For RHEL7, fix pyparsing require in systemtap.spec

* systemtap.spec: Fixed pyparsing requires issue for RHEL7. The package
  appears to have changed names.

When enabling python2 probes, make sure we find the right config program.

* configure.ac: For the python config program, look for 'python2-config'
  and 'python-config'.
* configure: Regenerated.

Remove C++11 string member function use for RHEL6.

* session.cxx (parse_cmdline): Remove C++11 string member function use for
  RHEL6. Functions include back(), front(), and push_back(), which can be
  easily replaced.
* setupdwfl.cxx (debuginfo_path_insert_sysroot): Ditto.
* tapsets.cxx (path_remove_sysroot): Ditto.
* util.cxx (get_distro_info): Ditto.
  (get_distro_info): Ditto.

Fix sysroot_sysenv.exp for RHEL6.

* testsuite/systemtap.base/sysroot_sysenv.exp: Fix test for RHEL6 by bind
  mounting /usr/src/kernels down into the fake sysroot.

Add lots more sysroot testing to systemtap.base/sysroot_sysenv.exp.

* testsuite/systemtap.base/sysroot_sysenv.exp: Add tests based on the 6
  sysroot patches sent in by Victor Kamensky. These will hopefully keep
  the sysroot functionality working. The tests use a custom mount
  namespace to "move" the system's /usr/lib/debug and /lib/modules
  directories into a sysroot directory.

sysroot: handle symbolic links with absolute name relative to sysroot

In case of symbolic link found under sysroot point to absolute path,
instead of trying to look for such absolute path in host system,
apply sysroot prefix first.

Corrected shebang on testsuite/runok/regex_noelide.stp

Corrected file permissions on testsuite/runok/regex_noelide.stp

BPF translator: array elements behave as if preinitialized to 0

Previously we exited a probe if there was an attempt to use an element
that an array did not contain. This does not match the behavior of the
default runtime which acts as though the element was preinitialized to
0. This commit changes the translator to match this behavior.

bpf-translate.cxx (visit_arrayindex): If map_lookup_elem returns 0,
set result to 0 instead of jumping to exit_block.

bpf_tests/array_preinit.stp: tests this change.

_stp_umodule_relocate needs target file path, not host file path

Strip of sysroot from module name is required when _stp_umodule_relocate
call is generated. Otherwise path won't match path on target and could
will fail to calculated address within the file.

configure: add option --enable-docs=prebuilt

Added a couple of small sysroot fixes.

* tapsets.cxx (dwarf_builder::build): Fix commit 4ffecddf5.
  (path_remove_sysroot): Fix extra '/' present at start of paths.

Delay adding sysroot path to module name in case of non absolute executable

Current stap code adds sysroot prematurely for probes that specify non
absolute path name, i.e like "foo", so when find_executable called it
receives full path as <sysroot>/foo and find_executable does not search
PATH while applying sysroot.

Fix delays adding sysroot till path inside of sysroot is searched first.

Also fix missing sysroot addition in glob expansion case.

session.cxx: note 4.16-rc4 passes stap smoke tests

stap-prep: prefer dnf to yum

Use dnf and dnf debuginfo-install if available, since these understand
some newer rpm constraints than yum does.  e.g., "or" constraints
on rawhide kernels.

sysroot: fix short release -r option handling

* setupdwfl.cxx (debuginfo_path_insert_sysroot): Add a '/' to the end of
  the sysroot for path_insert_sysroot().
  (setup_dwfl_kernel): Simplify logic when finding the kernel path to send
  to elfutils.

Make sure sysroot paths don't end with a slash.

* session.cxx (parse_cmdline): Make sure a sysroot path does not end with
  a '/', since we build paths like: sysroot + "/lib/modules". If the
  sysroot path ends with a '/', we end up with paths like
  '/SYSROOT//lib/modules'.
  (setup_kernel_release): Take a string parameter, not a character pointer.
* session.h: Update setup_kernel_release() prototype.

debuginfo lookup with sysroot case do not remove sysroot from file_name

If sysroot option is passed, and debug symbols reside in sysroot along
with executable <foo> in <foo_dir>/.debug/<foo_file> directory,
stap fails to find debuginfo because it strips out sysroot path
from file_name so dwfl_standard_find_debuginfo ends up looking at
host <foo_dir>/.debug/<foo_file> rather then checking
<sysroot>/<foo_dir>/.debug/<foo_file>.

Improve BPF tracepoint probe derivation

One goal of BPF tracepoint probe derivation is to determine the offsets of the members
of the BPF tracepoint context structs. These members are the probe's context variables.
The context structs always contain 8 bytes of padding before their first member.

During the probe derivation process, the context structs are replicated in the tracequery
modules and we read the module's debuginfo in order to determine the offsets of the
struct's members. Previously, the 8 bytes of padding was not included in the tracequery
structs and instead 8 was added to the offsets of members of these structs. This approach
ignores the possibility that these 8 bytes of padding could influence the amount of
padding between the struct's other members. This would result in incorrect offsets.

This patch adds an 8 byte member to the front of each of the tracequery structs to ensure
they are proper replicas of the tracepoint context structs.

PR22193 :: test that regex match (=~) is not elided (needs fixing)

misc PR15065 fix :: remove redundant cleanup code

* stapregex.cxx (stapdfa::emit_declaration): Remove redundant code.
* stapregex-dfa.cxx (dfa::emit): Make sure to clean up to STAPREGEX_MAX_TAG.
* testsuite/runok/regex_grouping.stp: Add testcase.

BPF tapset: fix gid()

* context.stp: replace gid()'s left shift with a logical right shift.

Merge branch 'master' of ssh://sourceware.org/git/systemtap

Use sysroot when looking for the System.map file.

Add sysroot to system_map_path + "/boot/System.map" case. Otherwise
stap tries to look symbols on host system and it produce error like this:

> Kernel symbol table /boot/System.map-4.9.78-yocto-standard unavailable, (No such file or directory)

BPF tapset: use userspace naming conventions for pid(), tid()

* context.stp: Undo naming changes made in commit 2c0ba52a181ec.
  pid() now returns the userspace pid of the target process instead
  of its kernel pid. tid() now returns the userspace tid of the target
  process instead of its kernel tgid.

Build fix regression on systems without nss.

After "eb3896ef0 Add the capability to list http servers." and
"3580be162 Move the code getting a certificate from a server into
each backend." commits on a system where nss-devel is not installed
or one build systemtap with --without-nss configure option, build
fails. Fix is simple add '#if HAVE_NSS' check around new code.

BPF tapset: fix pid(), tid()

* context.stp: BPF_FUNC_get_current_pid_tgid returns tgid << 32 | pid,
  so swap the names of pid() and tid(). Also replace tid()'s left shift
  with a logical right shift.

BPF translator: add 'in' operator for arrays.

* bpf-translate.cxx (bpf_unparser::visit_array_in): add support for
  the 'in' operator.

* bpf_tests/array_in.stp: tests these changes.

Fix PR22857 by updating vfs.do_mpage_readpage.

* tapset/linux/vfs.stp (vfs.do_mpage_readpage): Add support for ext4,
  which uses its own function (ext4_mpage_readpages).

Fix BZ1546179 by being careful when accessing sk_buff structures.

* tapset/linux/networking.stp: Fixed BZ1546179 by putting try/catch around
  sk_buff structure accesses (because the sk_buff structure has become
  quite overloaded with sharing). See block comment (or BZ1546179) for
  more details.
* tapset/linux/ipmib.stp: Ditto.
* tapset/linux/netfilter.stp: Ditto.
* tapset/linux/tcp.stp: Ditto.
* tapset/linux/tcpmib.stp: Ditto.

BPF translator: Add register spilling

* bpf-base.cxx (program::program): Initialize max_tmp_space.

* bpf-internal.h: Add MAX_BPF_STACK and BPF_REG_SIZE macros.

* bpf-opt.cxx (reg_alloc): Add support for spilling. Move various pieces
  of code into seperate functions for improved readability (finalize_allocation,
  merge, merge_copies, allocate).
  (spill, choose_spill_reg): New functions for spilling.
  (pref_sort_reg::cmp): Now imposes a strict weak ordering (prevents
  undefined behavior when used with std::sort).

* bpf-translate.cxx (emit_store): Add calls to program::use_tmp_space.

* bpf-tests/reg_alloc2.stp: Increase number of spills and improve readability.

testsuite: Use /usr/bin/env everywhere.

bz5274.stp and bz5274.a.stp where still using /bin/env instead of
/usr/bin/env like every other testsuite file.

The latest coreutils in rawhide dropped old /bin/* provides.
https://bugzilla.redhat.com/show_bug.cgi?id=1549063

Add a minor tweak to testsuite/systemtap.stress/tapset_functions.exp.

* testsuite/systemtap.stress/tapset_functions.exp: Tweak the way stap is
  started so that it is run the same both times. Otherwise, it can get a
  different list of functions between the two invocations.

Add CentOS support to the http server.

* httpd/docker/centos.json: Add JSON config file for CentOS.
* util.cxx (get_distro_info): Handle a blank line in the /etc/os-release
  file properly. Handle quoted strings properly.
* httpd/docker/fedora_install_package.py: Add CentOS support, by fixing
  grabbing debuginfo for systems using yum.

session.cxx: bump supported kernel version latest to 4.15.3

Fix a web server memory leak.

* httpd/nss_funcs.cxx (nss_get_server_cert_info): Fix memory leak causing
  SEC_ERROR_BUSY when shutting down.

dwflpp::function_entrypc if entry or low pc isn't given use base or start.

On Fedora rawhide a kernel compiled with GCC8 sometimes has subprograms
without an entry or low pc attribute, but with a ranges attribute. In that
case assume the base address or first start address of the range is the
entry pc to set a probe on.

Remove %clean section

None of currently supported distributions need that.
Last one was EL5 which is EOL for a while.

Signed-off-by: Igor Gnatenko <ignatenkobrain@fedoraproject.org>

Remove BuildRoot definition

None of currently supported distributions need that.
It was needed last for EL5 which is EOL now

Signed-off-by: Igor Gnatenko <ignatenkobrain@fedoraproject.org>

Escape macros in %changelog

Reference: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/thread/Y2ZUKK2B7T2IKXPMODNF6HB2O5T5TS6H/
Signed-off-by: Igor Gnatenko <ignatenkobrain@fedoraproject.org>

Fix very old Requires

Signed-off-by: Igor Gnatenko <ignatenkobrain@fedoraproject.org>

Make a few small changes to handle Fedora's switch to python 3 as default.

* configure.ac: Prefer 'python2' over 'python' for the python2
  executable.
* configure: Regenerated.
* httpd/backends.cxx (generate_module): Be sure which version of python
  we're calling.
* httpd/docker/stap_build_docker_image.py: Remove python shebang line
  since httpd/backends.cxx knows the right version of python to call.
* httpd/docker/fedora_install_package.py: Add comment about why using
  '/usr/bin/python' is OK.

Fix the netdev.ioctl probe point for rawhide kernels.

* tapset/linux/networking.stp: Fix the netdev.ioctl probe alias for the
  4.16.0-0.rc1.git4.1.fc28.x86_64 kernel.

systemtap.spec: Update Python 2 dependency declarations to new packaging standards

man error::pass5 - mention stap -t

Add undefine __brp_mangle_shebangs

Added strncpy()/strncat() code correctness fixes.

* doc/SystemTap_Tapset_Reference/docproc.c: gcc version 8 (on rawhide)
  rightly complains that the following code is wrong:
char buf[PATH_MAX];
strncpy(buf, foo, PATH_MAX);
  The code should be:
strncpy(buf, foo, PATH_MAX - 1);
  Note that a similar problem occurs with strncat(). Fix these errors by
  either subtracing one from the buffer length or switching to strlcpy()
  instead of strncpy().
* runtime/task_finder_vma.c: Ditto.
* staplog.c: Ditto.
* staprun/ctl.c: Ditto.
* staprun/staprun.c: Ditto.
* stapvirt.c: Ditto.
* tapset/linux/ip.stp: Ditto.
* tapset/tokenize.stp: Ditto.
* tapset/uconversions.stp: Ditto.
* testsuite/buildok/cmdline01.stp: Ditto.
* testsuite/buildok/two.stp: Ditto.
* testsuite/systemtap.syscall/bind.c: Ditto.
* testsuite/systemtap.syscall/nfsservctl.c: Ditto.

Add a couple of http client/server json-c 0.13 fixes.

* client-http.cxx (unpack_response): Add a json-c 0.13 (from rawhide) fix.
* httpd/server.cxx (get_key_values): Ditto.

Fixes for gcc 8

* includes/sys/sdt.h (__SDT_COND_SIGNED):  Add CT, cast type argument

Author: Will Cohen <wcohen.redhat.com>

* stap-serverd.cxx (generate_mok, handleRequest, handle_connection):
  Catch format overflow

* translate.cxx (translate_pass): Use ref in catch.

Fix an server error message with odd spacing.

* nss-server-info.cxx (add_server_trust): Make sure we don't output an
  extra space in an error message.

Make sure the user who starts the http server trusts it as an SSL peer.

* httpd/nss_funcs.cxx (nss_init): Make sure the server certificate is
  automatically trusted by the user starting the server.