summaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* bus-proxy: allow empty arguments to UpdateActivationEnvironment()v224David Herrmann2015-07-311-2/+2
| | | | | | There is no reason to prevent empty argument lists on UpdateActivationEnvironment(). Make sure we don't fail, but still skip the call to pid1.
* Merge pull request #806 from dvdhrm/proxy-cookieLennart Poettering2015-07-311-9/+24
|\ | | | | bus-proxy: fix cookie namespacing
| * bus-proxy: fix cookie namespacingDavid Herrmann2015-07-311-9/+24
| | | | | | | | | | | | | | | | If we generate messages from within bus-proxyd, we really must make sure that we do not clutter the cookie-namespace of our client. The client has full control over it, so we cannot steal cookies from it. However, we can re-use the cookie the client used for our request. As long as we only send a single message, we're fine.
* | logind: release VT-positions when closing sessionsDavid Herrmann2015-07-311-0/+6
| | | | | | | | | | | | | | | | | | Make sure we release VT-positions when a session is closed. Otherwise, lingering sessions will occupy VTs and prevent next logins from succeeding. Note that we already release session-devices when closing a session, so there cannot be anyone using the VT anymore.
* | timesyncd: remove RLIMIT_NPROCKay Sievers2015-07-311-4/+0
|/ | | | | NSS plugins might create additional threads. Remove the limit, we cannot really make any assumptions here.
* Merge pull request #804 from dvdhrm/proxy-directed2Daniel Mack2015-07-313-18/+22
|\ | | | | bus-proxy: make NameAcquired/Lost directed (v2)
| * bus-proxy: don't filter directed signalsDavid Herrmann2015-07-311-14/+1
| | | | | | | | | | | | NameAcquired and NameLost are directed signals. Never ever filter them on dbus1. On dbus1, filters *exclusively* apply to broadcasts! Hence, we must always forward directed signals unconditionally!
| * bus-proxy: fix NameAcquired and NameLost to be directedDavid Herrmann2015-07-313-4/+21
| | | | | | | | | | | | The NameAcquired and NameLost signals are _directed_ signals. Make sure we properly set the destination correctly, and verify it in our proxy-test.
* | Merge pull request #805 from dvdhrm/proxy-activationDaniel Mack2015-07-313-12/+67
|\ \ | | | | | | bus-proxy: make StartServiceByName synchronous
| * | bus-proxy: make StartServiceByName synchronousDavid Herrmann2015-07-313-12/+67
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The StartServiceByName() call was provided by dbus-daemon to activate a service without sending a message. On receiption, dbus-daemon schedules an activation request (different modes are supported) and sends back the reply once activation is done. With kdbus, we marked StartServiceByName() as deprecated. There is no real reason to start services explicitly. Instead, applications should just *use* the service and rely on it being activated implicitly. However, we provide compatibility with dbus-daemon and implement StartServiceByName() on the proxy via a call to org.freedesktop.DBus.Peer.Ping() on the destination. This will activate the peer implicitly as part of the no-op Ping() method call (regardless whether the peer actually implements that call). Now, the problem is, StartServiceByName() was synchronous on dbus-daemon but isn't on bus-proxy. Hence, on return, there is no guarantee that ListNames includes the activated name. As this is required by some applications, we need to make this synchronous. This patch makes the proxy track the Ping() method call and send the reply of StartServiceByName() only once Ping() returned. We do not look at possible errors of Ping(), as there is no strict requirement for the peer to implement org.freedesktop.DBus.Peer. Furthermore, any interesting error should have already been caught by sd_bus_send() before. Note: This race was triggered by gdbus. The gdbus-proxy implementation relies on a name to be available after StartServiceByName() returns. This is highly fragile and should be dropped by gdbus. Even if the call is synchronous, there is no reason whatsoever to assume the service did not exit-on-idle before ListNames() returns. However, this race is much less likely than the startup race, so we try to be compatible to dbus-daemon now.
* | basic: fix error/memleak in socket-utilDavid Herrmann2015-07-311-1/+1
|/ | | | | | Probably a typo, checking 'ret' instead of the return value 'p'. This might cause the function to return failure, even though it succeeded. Furthermore, it might leak resources.
* systemd-boot: sort /EFI/Linux/*.efi filesKay Sievers2015-07-301-8/+21
|
* unit-name: fix memory leakDaniel Mack2015-07-301-1/+3
| | | | | Fix a memory leak introduced by 2fc09a9c. 's' is used twice in this function, and if free_and_strdup() fails, the pointer is left untouched.
* Merge pull request #787 from dvdhrm/bus-listDaniel Mack2015-07-302-26/+26
|\ | | | | sd-bus: fix parsing of KDBUS_CMD_LIST
| * sd-bus: fix parsing of KDBUS_CMD_LISTDavid Herrmann2015-07-302-26/+26
| | | | | | | | | | | | | | | | | | | | | | We *must not* assume that an entry returned by KDBUS_CMD_LIST only carries a single KDBUS_ITEM_OWNED_NAME. Similarly, we already parse multiple such items for message-metadata, so make sure we support the same on KDBUS_CMD_LIST. By relying on the kernel to return all names separately, we limit the kernel API significantly. Stop this and let the kernel decide how to return its data.
* | tree-wide: use free_and_strdup()Daniel Mack2015-07-3010-59/+40
| | | | | | | | | | Use free_and_strdup() where appropriate and replace equivalent, open-coded versions.
* | Merge pull request #779 from poettering/fflush-and-checkDaniel Mack2015-07-2925-214/+247
|\ \ | | | | | | tree-wide: port everything over to fflush_and_check()
| * | tree-wide: port everything over to fflush_and_check()Lennart Poettering2015-07-2925-214/+247
| |/ | | | | | | | | | | | | | | Some places invoked fflush() directly with their own manual error checking, let's unify all that by using fflush_and_check(). This also unifies the general error paths of fflush()+rename() file writers.
* | Merge pull request #764 from ssahani/vxlan1Tom Gundersen2015-07-297-15/+53
|\ \ | | | | | | networkd: Add VXLAN Netlink attributes
| * | networkd: vxlan add support for GBPSusant Sahani2015-07-293-1/+9
| | | | | | | | | | | | | | | | | | | | | This patch add support for vxlan VXLAN Group Policy Option. https://tools.ietf.org/html/draft-smith-vxlan-group-policy-00 http://lwn.net/Articles/628683/
| * | sd-netlink: introduce api for new NL type NLA_FLAGSusant Sahani2015-07-292-0/+19
| | |
| * | sd-netlink: add support for new type NETLINK_TYPE_FLAGSusant Sahani2015-07-291-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | NETLINK_TYPE_FLAG is NLA_FLAG. This new type will be used in NLA_FLAG for example IFLA_VXLAN_GBP and IFLA_VXLAN_REMCSUM_NOPARTIAL
| * | sd-netlink: add support for vxlan attributesSusant Sahani2015-07-291-14/+24
| | |
* | | Merge branch 'master' of github.com:systemd/systemdDaniel Mack2015-07-296-38/+85
|\ \ \
| * | | sd-boot: stub - export LoaderDevicePartUUIDKay Sievers2015-07-294-24/+82
| | | |
| * | | Merge pull request #778 from ssahani/flowDaniel Mack2015-07-292-14/+3
| |\ \ \ | | |_|/ | |/| | networkd: fix size of networkd binary
| | * | networkd: fix size of networkd binarySusant Sahani2015-07-292-14/+3
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes the size of networkd Bug #775 before: size systemd-networkd text data bss dec hex filename 1493755 8424392 2337 9920484 975fe4 systemd-networkd After $ size systemd-networkd text data bss dec hex filename 1493555 35752 2337 1531644 175efc systemd-networkd
* | | Merge branch 'pr/761'Daniel Mack2015-07-291-1/+7
|\ \ \ | |/ / |/| |
| * | networkd-wait-online: only consider interfaces given on the commandlineTom Gundersen2015-07-291-0/+6
| | | | | | | | | | | | If some interfaces are given on the commandline, ignore all others.
| * | networkd-wait-online: fix raceTom Gundersen2015-07-291-1/+1
| | | | | | | | | | | | | | | | | | We must consider 'pending' links as if they may be managed by networkd, as this is the state we enter before deciding wether networkd should manage the link or not, so we better wait for this decision being made.
* | | Merge pull request #777 from kaysievers/gtp-autoLennart Poettering2015-07-293-213/+268
|\ \ \ | | | | | | | | gpt-auto-generator: merge efi-boot-generator
| * | | gpt-auto-generator: merge efi-boot-generatorKay Sievers2015-07-293-213/+268
| | | |
* | | | Merge pull request #774 from dvdhrm/gvariant3Lennart Poettering2015-07-294-26/+64
|\ \ \ \ | |/ / / |/| | | gvariant: fix encoding of unary type and root-variant (v3)
| * | | sd-bus: fix encoding/decoding gvariant root containerDavid Herrmann2015-07-291-7/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The gvariant root container contains a 'variant' at the end, which embeds the whole message body. This variant *must* contain a structure so we are compatible to dbus1. Otherwise, it could encode at most 1 type, instead of a full signature. Our gvariant message parser already parses the variant-content as a structure, so we're mostly good. However, it does *not* include the opening and closing parantheses, nor does it parse them. This patch fixes the decoder to verify a message contains the parantheses, and also make the encoder add those parantheses into the marshaled message.
| * | | sd-bus: fix marshaling of unary typeDavid Herrmann2015-07-293-11/+38
| | | | | | | | | | | | | | | | | | | | The unary type has a fixed size of 1 in gvariant. Make sure we properly encode it as such. Right now, we encode/decode it as empty sequence.
| * | | sd-bus: drop redundant codeDavid Herrmann2015-07-291-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | If c->item_size is 0, the next item to parse in a structure is empty. However, this also implies that the signature must be empty. The latter case is already handled just fine by enter_struct_or_dict_entry() so there is no reason to handle the same case in the caller.
| * | | sd-bus: don't assert() on valid signaturesDavid Herrmann2015-07-292-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Right now sd_bus_message_skip() will abort execution if passed a signature of the unary type "()". Regardless whether this should be supported or not, we really must not abort. Drop the incorrect assertion and add a test-case for this.
* | | | Merge pull request #769 from poettering/socket-util-no-logDaniel Mack2015-07-292-3/+4
|\ \ \ \ | |/ / / |/| | | socket-util: library calls shouldn't log on their own
| * | | socket-util: library calls shouldn't log on their ownLennart Poettering2015-07-292-3/+4
| | | | | | | | | | | | | | | | Instead, make sure that all callers log properly.
* | | | Merge pull request #768 from poettering/resolved-localhostDaniel Mack2015-07-292-0/+7
|\ \ \ \ | | | | | | | | | | resolved: never attempt to resolve loopback addresses via DNS/LLMNR/mDNS
| * | | | resolved: never attempt to resolve loopback addresses via DNS/LLMNR/mDNSLennart Poettering2015-07-292-0/+7
| |/ / / | | | | | | | | | | | | | | | | We already refuse to resolve "localhost", hence we should also refuse resolving "127.0.0.1" and friends.
* | | | treewide: fix typosTorstein Husebø2015-07-291-1/+1
| | | |
* | | | Merge pull request #747 from dvdhrm/consoledDavid Herrmann2015-07-2937-24252/+0
|\ \ \ \ | |_|_|/ |/| | | terminal: drop unfinished code
| * | | terminal: drop unfinished codeDavid Herrmann2015-07-2737-24252/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This drops the libsystemd-terminal and systemd-consoled code for various reasons: * It's been sitting there unfinished for over a year now and won't get finished any time soon. * Since its initial creation, several parts need significant rework: The input handling should be replaced with the now commonly used libinput, the drm accessors should coordinate the handling of mode-object hotplugging (including split connectors) with other DRM users, and the internal library users should be converted to sd-device and friends. * There is still significant kernel work required before sd-console is really useful. This includes, but is not limited to, simpledrm and drmlog. * The authority daemon is needed before all this code can be used for real. And this will definitely take a lot more time to get done as no-one else is currently working on this, but me. * kdbus maintenance has taken up way more time than I thought and it has much higher priority. I don't see me spending much time on the terminal code in the near future. If anyone intends to hack on this, please feel free to contact me. I'll gladly help you out with any issues. Once kdbus and authorityd are finished (whenever that will be..) I'll definitely pick this up again. But until then, lets reduce compile times and maintenance efforts on this code and drop it for now.
* | | | resolved: packet - refuse empty type bitmapsTom Gundersen2015-07-281-2/+6
| | | | | | | | | | | | | | | | | | | | The NSEC type itself must at least be in the bitmap, so NSEC records with empty bitmaps must be bogus.
* | | | resolved: packet - fix read_type_window()Tom Gundersen2015-07-281-1/+2
| | | | | | | | | | | | | | | | We were tracking the bit offset inside each byte, rather than inside the whole bitmap.
* | | | resolve: packet - fix append_types()Tom Gundersen2015-07-281-24/+16
| |_|/ |/| | | | | | | | We were counting the number of bits set rather than the number of bytes they occupied.
* | | Merge pull request #758 from poettering/dns-is-equalTom Gundersen2015-07-283-2/+44
|\ \ \ | | | | | | | | resolved: compare dns question arrays properly
| * | | resolved: compare dns question arrays properlyLennart Poettering2015-07-283-2/+44
| | |/ | |/| | | | | | | | | | | | | Let's optimize things a bit and properly compare DNS question arrays, instead of checking if they are mutual supersets. This also makes ANY query handling more accurate.
* | | Merge pull request #732 from ssahani/macvtapTom Gundersen2015-07-289-3/+35
|\ \ \ | | | | | | | | networkd: add support for Macvtap
View Lorry Controller Status