Fix setuid by setting %attr on cockpit-polkit

Upstream bug URL: https://bugzilla.redhat.com/show_bug.cgi?id=117858

Signed-off-by: Stef Walter <stefw@redhat.com>
 * Added comment to spec file
test: Fix bad syntax in previous commit
agent: Fix another use after free in CockpitDBusJson
agent: Fork CockpitDBusJson into the old CockpitDBusJson1

We'll be implementing a new dbus-json2 protocol and migrating
away from dbus-json1

The javascript code which calls into the old protocol is rather
ugly, requiring explicit use of { protocol: "dbus-json1" }
everywhere. That's intentional.

The first change to the new dbus-json2 is that we include the
signature along with variants like this:

 { "sig": "u", "val": 1234 }

Closes: #568