X-Git-Url: http://git.liburcu.org/?a=blobdiff_plain;f=src%2Flttng-abi.c;h=219d20a044e284c63f1688a9a06df514b28bceb6;hb=badfe9f5c396efb1b00e5f2abcded2e4ac4a5bac;hp=4dd6e7e5e2fff77290f493664b543e66471b0a96;hpb=92143b2c56562f525bc74777a1ef9e320e4a5942;p=lttng-modules.git

diff --git a/src/lttng-abi.c b/src/lttng-abi.c
index 4dd6e7e5..219d20a0 100644
--- a/src/lttng-abi.c
+++ b/src/lttng-abi.c
@@ -1080,7 +1080,7 @@ int lttng_metadata_ring_buffer_open(struct inode *inode, struct file *file)
 	 * session, we need to keep our own reference on the transport.
 	 */
 	if (!try_module_get(stream->transport->owner)) {
-		printk(KERN_WARNING "LTT : Can't lock transport module.\n");
+		printk(KERN_WARNING "LTTng: Can't lock transport module.\n");
 		return -EBUSY;
 	}
 	return lib_ring_buffer_open(inode, file, buf);
@@ -1234,7 +1234,7 @@ int lttng_abi_open_metadata_stream(struct file *channel_file)
 	 * session, we need to keep our own reference on the transport.
 	 */
 	if (!try_module_get(metadata_stream->transport->owner)) {
-		printk(KERN_WARNING "LTT : Can't lock transport module.\n");
+		printk(KERN_WARNING "LTTng: Can't lock transport module.\n");
 		ret = -EINVAL;
 		goto notransport;
 	}
@@ -1266,6 +1266,46 @@ nomem:
 	return ret;
 }
 
+static
+int lttng_abi_validate_event_param(struct lttng_kernel_event *event_param)
+{
+	/* Limit ABI to implemented features. */
+	switch (event_param->instrumentation) {
+	case LTTNG_KERNEL_SYSCALL:
+		switch (event_param->u.syscall.entryexit) {
+		case LTTNG_KERNEL_SYSCALL_ENTRYEXIT:
+			break;
+		default:
+			return -EINVAL;
+		}
+		switch (event_param->u.syscall.abi) {
+		case LTTNG_KERNEL_SYSCALL_ABI_ALL:
+			break;
+		default:
+			return -EINVAL;
+		}
+		switch (event_param->u.syscall.match) {
+		case LTTNG_SYSCALL_MATCH_NAME:
+			break;
+		default:
+			return -EINVAL;
+		}
+		break;
+
+	case LTTNG_KERNEL_TRACEPOINT:	/* Fallthrough */
+	case LTTNG_KERNEL_KPROBE:	/* Fallthrough */
+	case LTTNG_KERNEL_KRETPROBE:	/* Fallthrough */
+	case LTTNG_KERNEL_NOOP:		/* Fallthrough */
+	case LTTNG_KERNEL_UPROBE:
+		break;
+
+	case LTTNG_KERNEL_FUNCTION:	/* Fallthrough */
+	default:
+		return -EINVAL;
+	}
+	return 0;
+}
+
 static
 int lttng_abi_create_event(struct file *channel_file,
 			   struct lttng_kernel_event *event_param)
@@ -1307,6 +1347,9 @@ int lttng_abi_create_event(struct file *channel_file,
 		ret = -EOVERFLOW;
 		goto refcount_error;
 	}
+	ret = lttng_abi_validate_event_param(event_param);
+	if (ret)
+		goto event_error;
 	if (event_param->instrumentation == LTTNG_KERNEL_TRACEPOINT
 			|| event_param->instrumentation == LTTNG_KERNEL_SYSCALL) {
 		struct lttng_enabler *enabler;
@@ -2018,7 +2061,7 @@ int __init lttng_abi_init(void)
 					&lttng_proc_ops, NULL);
 
 	if (!lttng_proc_dentry) {
-		printk(KERN_ERR "Error creating LTTng control file\n");
+		printk(KERN_ERR "LTTng: Error creating control file\n");
 		ret = -ENOMEM;
 		goto error;
 	}