*
* Integers read/write functions.
*
- * Copyright 2010 - Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
+ * Copyright 2010-2011 EfficiOS Inc. and Linux Foundation
+ *
+ * Author: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
*
* Permission is hereby granted, free of charge, to any person obtaining a copy
* of this software and associated documentation files (the "Software"), to deal
*
* The above copyright notice and this permission notice shall be included in
* all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
*/
#include <babeltrace/ctf/types.h>
#include <babeltrace/bitfield.h>
#include <stdint.h>
#include <glib.h>
-#include <endian.h>
+#include <babeltrace/endian.h>
/*
* The aligned read/write functions are expected to be faster than the
*/
static
-void _aligned_integer_read(struct stream_pos *ppos,
- struct definition *definition)
+int _aligned_integer_read(struct stream_pos *ppos,
+ struct definition *definition)
{
struct definition_integer *integer_definition =
container_of(definition, struct definition_integer, p);
int rbo = (integer_declaration->byte_order != BYTE_ORDER); /* reverse byte order */
ctf_align_pos(pos, integer_declaration->p.alignment);
- assert(!(pos->offset % CHAR_BIT));
+ if (!ctf_pos_access_ok(pos, integer_declaration->len))
+ return -EFAULT;
+
+ assert(!(pos->offset % CHAR_BIT));
if (!integer_declaration->signedness) {
switch (integer_declaration->len) {
case 8:
v = *(const int16_t *) ctf_get_pos_addr(pos);
integer_definition->value._signed =
- rbo ? GUINT16_SWAP_LE_BE(v) : v;
+ rbo ? (int16_t) GUINT16_SWAP_LE_BE(v) : v;
break;
}
case 32:
v = *(const int32_t *) ctf_get_pos_addr(pos);
integer_definition->value._signed =
- rbo ? GUINT32_SWAP_LE_BE(v) : v;
+ rbo ? (int32_t) GUINT32_SWAP_LE_BE(v) : v;
break;
}
case 64:
v = *(const int64_t *) ctf_get_pos_addr(pos);
integer_definition->value._signed =
- rbo ? GUINT64_SWAP_LE_BE(v) : v;
+ rbo ? (int64_t) GUINT64_SWAP_LE_BE(v) : v;
break;
}
default:
}
}
ctf_move_pos(pos, integer_declaration->len);
+ return 0;
}
static
-void _aligned_integer_write(struct stream_pos *ppos,
+int _aligned_integer_write(struct stream_pos *ppos,
struct definition *definition)
{
struct definition_integer *integer_definition =
int rbo = (integer_declaration->byte_order != BYTE_ORDER); /* reverse byte order */
ctf_align_pos(pos, integer_declaration->p.alignment);
- assert(!(pos->offset % CHAR_BIT));
+ if (!ctf_pos_access_ok(pos, integer_declaration->len))
+ return -EFAULT;
+
+ assert(!(pos->offset % CHAR_BIT));
if (pos->dummy)
goto end;
if (!integer_declaration->signedness) {
break;
case 16:
*(int16_t *) ctf_get_pos_addr(pos) = rbo ?
- GUINT16_SWAP_LE_BE((int16_t) v) :
+ (int16_t) GUINT16_SWAP_LE_BE((int16_t) v) :
(int16_t) v;
break;
case 32:
*(int32_t *) ctf_get_pos_addr(pos) = rbo ?
- GUINT32_SWAP_LE_BE((int32_t) v) :
+ (int32_t) GUINT32_SWAP_LE_BE((int32_t) v) :
(int32_t) v;
break;
case 64:
}
end:
ctf_move_pos(pos, integer_declaration->len);
+ return 0;
}
-void ctf_integer_read(struct stream_pos *ppos, struct definition *definition)
+int ctf_integer_read(struct stream_pos *ppos, struct definition *definition)
{
struct definition_integer *integer_definition =
container_of(definition, struct definition_integer, p);
if (!(integer_declaration->p.alignment % CHAR_BIT)
&& !(integer_declaration->len % CHAR_BIT)) {
- _aligned_integer_read(ppos, definition);
- return;
+ return _aligned_integer_read(ppos, definition);
}
ctf_align_pos(pos, integer_declaration->p.alignment);
+
+ if (!ctf_pos_access_ok(pos, integer_declaration->len))
+ return -EFAULT;
+
if (!integer_declaration->signedness) {
if (integer_declaration->byte_order == LITTLE_ENDIAN)
- bt_bitfield_read_le(pos->base, unsigned long,
+ bt_bitfield_read_le(mmap_align_addr(pos->base_mma) +
+ pos->mmap_base_offset, unsigned long,
pos->offset, integer_declaration->len,
&integer_definition->value._unsigned);
else
- bt_bitfield_read_be(pos->base, unsigned long,
+ bt_bitfield_read_be(mmap_align_addr(pos->base_mma) +
+ pos->mmap_base_offset, unsigned long,
pos->offset, integer_declaration->len,
&integer_definition->value._unsigned);
} else {
if (integer_declaration->byte_order == LITTLE_ENDIAN)
- bt_bitfield_read_le(pos->base, unsigned long,
+ bt_bitfield_read_le(mmap_align_addr(pos->base_mma) +
+ pos->mmap_base_offset, unsigned long,
pos->offset, integer_declaration->len,
&integer_definition->value._signed);
else
- bt_bitfield_read_be(pos->base, unsigned long,
+ bt_bitfield_read_be(mmap_align_addr(pos->base_mma) +
+ pos->mmap_base_offset, unsigned long,
pos->offset, integer_declaration->len,
&integer_definition->value._signed);
}
ctf_move_pos(pos, integer_declaration->len);
+ return 0;
}
-void ctf_integer_write(struct stream_pos *ppos, struct definition *definition)
+int ctf_integer_write(struct stream_pos *ppos, struct definition *definition)
{
struct definition_integer *integer_definition =
container_of(definition, struct definition_integer, p);
if (!(integer_declaration->p.alignment % CHAR_BIT)
&& !(integer_declaration->len % CHAR_BIT)) {
- _aligned_integer_write(ppos, definition);
- return;
+ return _aligned_integer_write(ppos, definition);
}
ctf_align_pos(pos, integer_declaration->p.alignment);
+
+ if (!ctf_pos_access_ok(pos, integer_declaration->len))
+ return -EFAULT;
+
if (pos->dummy)
goto end;
if (!integer_declaration->signedness) {
if (integer_declaration->byte_order == LITTLE_ENDIAN)
- bt_bitfield_write_le(pos->base, unsigned long,
+ bt_bitfield_write_le(mmap_align_addr(pos->base_mma) +
+ pos->mmap_base_offset, unsigned long,
pos->offset, integer_declaration->len,
integer_definition->value._unsigned);
else
- bt_bitfield_write_be(pos->base, unsigned long,
+ bt_bitfield_write_be(mmap_align_addr(pos->base_mma) +
+ pos->mmap_base_offset, unsigned long,
pos->offset, integer_declaration->len,
integer_definition->value._unsigned);
} else {
if (integer_declaration->byte_order == LITTLE_ENDIAN)
- bt_bitfield_write_le(pos->base, unsigned long,
+ bt_bitfield_write_le(mmap_align_addr(pos->base_mma) +
+ pos->mmap_base_offset, unsigned long,
pos->offset, integer_declaration->len,
integer_definition->value._signed);
else
- bt_bitfield_write_be(pos->base, unsigned long,
+ bt_bitfield_write_be(mmap_align_addr(pos->base_mma) +
+ pos->mmap_base_offset, unsigned long,
pos->offset, integer_declaration->len,
integer_definition->value._signed);
}
end:
ctf_move_pos(pos, integer_declaration->len);
+ return 0;
}
projects / babeltrace.git / blobdiff