diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2012-04-16 17:14:06 +1000 |
|---|---|---|
| committer | Andrew Bartlett <abartlet@samba.org> | 2012-04-18 12:05:00 +1000 |
| commit | 492942a66ad89172135512cd433b50c13726f777 (patch) | |
| tree | bd609f41480fb71ca51eb9add04a5d386eba1735 /source3/lib | |
| parent | 351c78fc670d5e26ebc2deea7ef930a94b116697 (diff) | |
| download | samba-492942a66ad89172135512cd433b50c13726f777.tar.gz samba-492942a66ad89172135512cd433b50c13726f777.tar.bz2 samba-492942a66ad89172135512cd433b50c13726f777.zip | |
s3-xattr_tdb: Be nice to xattr_tdb_getxattr callers, return the full blob
In this case, the blob is already in memory, so it is easier to return the full
blob to the caller, and let the caller decide if some interface restriction
stops the full blob from being passed all the way up the stack.
This allows us to quickly write a python wrapper for this xattr storage
mechanism.
Andrew Bartlett
Diffstat (limited to 'source3/lib')
| -rw-r--r-- | source3/lib/xattr_tdb.c | 12 | ||||
| -rw-r--r-- | source3/lib/xattr_tdb.h | 3 |
2 files changed, 6 insertions, 9 deletions
diff --git a/source3/lib/xattr_tdb.c b/source3/lib/xattr_tdb.c index 21223f04b8..6f33163ed6 100644 --- a/source3/lib/xattr_tdb.c +++ b/source3/lib/xattr_tdb.c @@ -169,8 +169,9 @@ static NTSTATUS xattr_tdb_save_attrs(struct db_record *rec, */ ssize_t xattr_tdb_getattr(struct db_context *db_ctx, + TALLOC_CTX *mem_ctx, const struct file_id *id, - const char *name, void *value, size_t size) + const char *name, DATA_BLOB *blob) { struct tdb_xattrs *attribs; uint32_t i; @@ -200,13 +201,8 @@ ssize_t xattr_tdb_getattr(struct db_context *db_ctx, goto fail; } - if (attribs->eas[i].value.length > size) { - errno = ERANGE; - goto fail; - } - - memcpy(value, attribs->eas[i].value.data, - attribs->eas[i].value.length); + *blob = attribs->eas[i].value; + talloc_steal(mem_ctx, blob->data); result = attribs->eas[i].value.length; fail: diff --git a/source3/lib/xattr_tdb.h b/source3/lib/xattr_tdb.h index 0a54833f51..03bc43a38c 100644 --- a/source3/lib/xattr_tdb.h +++ b/source3/lib/xattr_tdb.h @@ -26,8 +26,9 @@ /* The following definitions come from lib/util/xattr_tdb.c */ ssize_t xattr_tdb_getattr(struct db_context *db_ctx, + TALLOC_CTX *mem_ctx, const struct file_id *id, - const char *name, void *value, size_t size); + const char *name, DATA_BLOB *blob); int xattr_tdb_setattr(struct db_context *db_ctx, const struct file_id *id, const char *name, const void *value, size_t size, int flags); |