g_new(T, n) is neater than g_malloc(sizeof(T) * n).  It's also safer,
for two reasons.  One, it catches multiplication overflowing size_t.
Two, it returns T * rather than void *, which lets the compiler catch
more type errors.
This commit only touches allocations with size arguments of the form
sizeof(T).
Patch created mechanically with:
    $ spatch --in-place --sp-file scripts/coccinelle/use-g_new-etc.cocci \
	     --macro-file scripts/cocci-macro-file.h FILES...
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Acked-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <20220315144156.1595462-4-armbru@redhat.com>
Reviewed-by: Pavel Dovgalyuk <Pavel.Dovgalyuk@ispras.ru>
		
	
			
		
			
				
	
	
		
			256 lines
		
	
	
		
			6.7 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
			
		
		
	
	
			256 lines
		
	
	
		
			6.7 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
/*
 | 
						|
 * QEMU Crypto Device Implementation
 | 
						|
 *
 | 
						|
 * Copyright (c) 2016 HUAWEI TECHNOLOGIES CO., LTD.
 | 
						|
 *
 | 
						|
 * Authors:
 | 
						|
 *    Gonglei <arei.gonglei@huawei.com>
 | 
						|
 *
 | 
						|
 * This library is free software; you can redistribute it and/or
 | 
						|
 * modify it under the terms of the GNU Lesser General Public
 | 
						|
 * License as published by the Free Software Foundation; either
 | 
						|
 * version 2.1 of the License, or (at your option) any later version.
 | 
						|
 *
 | 
						|
 * This library is distributed in the hope that it will be useful,
 | 
						|
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 | 
						|
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 | 
						|
 * Lesser General Public License for more details.
 | 
						|
 *
 | 
						|
 * You should have received a copy of the GNU Lesser General Public
 | 
						|
 * License along with this library; if not, see <http://www.gnu.org/licenses/>.
 | 
						|
 *
 | 
						|
 */
 | 
						|
 | 
						|
#include "qemu/osdep.h"
 | 
						|
#include "sysemu/cryptodev.h"
 | 
						|
#include "qapi/error.h"
 | 
						|
#include "qapi/visitor.h"
 | 
						|
#include "qemu/config-file.h"
 | 
						|
#include "qom/object_interfaces.h"
 | 
						|
#include "hw/virtio/virtio-crypto.h"
 | 
						|
 | 
						|
 | 
						|
static QTAILQ_HEAD(, CryptoDevBackendClient) crypto_clients;
 | 
						|
 | 
						|
 | 
						|
CryptoDevBackendClient *
 | 
						|
cryptodev_backend_new_client(const char *model,
 | 
						|
                                    const char *name)
 | 
						|
{
 | 
						|
    CryptoDevBackendClient *cc;
 | 
						|
 | 
						|
    cc = g_new0(CryptoDevBackendClient, 1);
 | 
						|
    cc->model = g_strdup(model);
 | 
						|
    if (name) {
 | 
						|
        cc->name = g_strdup(name);
 | 
						|
    }
 | 
						|
 | 
						|
    QTAILQ_INSERT_TAIL(&crypto_clients, cc, next);
 | 
						|
 | 
						|
    return cc;
 | 
						|
}
 | 
						|
 | 
						|
void cryptodev_backend_free_client(
 | 
						|
                  CryptoDevBackendClient *cc)
 | 
						|
{
 | 
						|
    QTAILQ_REMOVE(&crypto_clients, cc, next);
 | 
						|
    g_free(cc->name);
 | 
						|
    g_free(cc->model);
 | 
						|
    g_free(cc->info_str);
 | 
						|
    g_free(cc);
 | 
						|
}
 | 
						|
 | 
						|
void cryptodev_backend_cleanup(
 | 
						|
             CryptoDevBackend *backend,
 | 
						|
             Error **errp)
 | 
						|
{
 | 
						|
    CryptoDevBackendClass *bc =
 | 
						|
                  CRYPTODEV_BACKEND_GET_CLASS(backend);
 | 
						|
 | 
						|
    if (bc->cleanup) {
 | 
						|
        bc->cleanup(backend, errp);
 | 
						|
    }
 | 
						|
}
 | 
						|
 | 
						|
int64_t cryptodev_backend_sym_create_session(
 | 
						|
           CryptoDevBackend *backend,
 | 
						|
           CryptoDevBackendSymSessionInfo *sess_info,
 | 
						|
           uint32_t queue_index, Error **errp)
 | 
						|
{
 | 
						|
    CryptoDevBackendClass *bc =
 | 
						|
                      CRYPTODEV_BACKEND_GET_CLASS(backend);
 | 
						|
 | 
						|
    if (bc->create_session) {
 | 
						|
        return bc->create_session(backend, sess_info, queue_index, errp);
 | 
						|
    }
 | 
						|
 | 
						|
    return -1;
 | 
						|
}
 | 
						|
 | 
						|
int cryptodev_backend_sym_close_session(
 | 
						|
           CryptoDevBackend *backend,
 | 
						|
           uint64_t session_id,
 | 
						|
           uint32_t queue_index, Error **errp)
 | 
						|
{
 | 
						|
    CryptoDevBackendClass *bc =
 | 
						|
                      CRYPTODEV_BACKEND_GET_CLASS(backend);
 | 
						|
 | 
						|
    if (bc->close_session) {
 | 
						|
        return bc->close_session(backend, session_id, queue_index, errp);
 | 
						|
    }
 | 
						|
 | 
						|
    return -1;
 | 
						|
}
 | 
						|
 | 
						|
static int cryptodev_backend_sym_operation(
 | 
						|
                 CryptoDevBackend *backend,
 | 
						|
                 CryptoDevBackendSymOpInfo *op_info,
 | 
						|
                 uint32_t queue_index, Error **errp)
 | 
						|
{
 | 
						|
    CryptoDevBackendClass *bc =
 | 
						|
                      CRYPTODEV_BACKEND_GET_CLASS(backend);
 | 
						|
 | 
						|
    if (bc->do_sym_op) {
 | 
						|
        return bc->do_sym_op(backend, op_info, queue_index, errp);
 | 
						|
    }
 | 
						|
 | 
						|
    return -VIRTIO_CRYPTO_ERR;
 | 
						|
}
 | 
						|
 | 
						|
int cryptodev_backend_crypto_operation(
 | 
						|
                 CryptoDevBackend *backend,
 | 
						|
                 void *opaque,
 | 
						|
                 uint32_t queue_index, Error **errp)
 | 
						|
{
 | 
						|
    VirtIOCryptoReq *req = opaque;
 | 
						|
 | 
						|
    if (req->flags == CRYPTODEV_BACKEND_ALG_SYM) {
 | 
						|
        CryptoDevBackendSymOpInfo *op_info;
 | 
						|
        op_info = req->u.sym_op_info;
 | 
						|
 | 
						|
        return cryptodev_backend_sym_operation(backend,
 | 
						|
                         op_info, queue_index, errp);
 | 
						|
    } else {
 | 
						|
        error_setg(errp, "Unsupported cryptodev alg type: %" PRIu32 "",
 | 
						|
                   req->flags);
 | 
						|
       return -VIRTIO_CRYPTO_NOTSUPP;
 | 
						|
    }
 | 
						|
 | 
						|
    return -VIRTIO_CRYPTO_ERR;
 | 
						|
}
 | 
						|
 | 
						|
static void
 | 
						|
cryptodev_backend_get_queues(Object *obj, Visitor *v, const char *name,
 | 
						|
                             void *opaque, Error **errp)
 | 
						|
{
 | 
						|
    CryptoDevBackend *backend = CRYPTODEV_BACKEND(obj);
 | 
						|
    uint32_t value = backend->conf.peers.queues;
 | 
						|
 | 
						|
    visit_type_uint32(v, name, &value, errp);
 | 
						|
}
 | 
						|
 | 
						|
static void
 | 
						|
cryptodev_backend_set_queues(Object *obj, Visitor *v, const char *name,
 | 
						|
                             void *opaque, Error **errp)
 | 
						|
{
 | 
						|
    CryptoDevBackend *backend = CRYPTODEV_BACKEND(obj);
 | 
						|
    uint32_t value;
 | 
						|
 | 
						|
    if (!visit_type_uint32(v, name, &value, errp)) {
 | 
						|
        return;
 | 
						|
    }
 | 
						|
    if (!value) {
 | 
						|
        error_setg(errp, "Property '%s.%s' doesn't take value '%" PRIu32 "'",
 | 
						|
                   object_get_typename(obj), name, value);
 | 
						|
        return;
 | 
						|
    }
 | 
						|
    backend->conf.peers.queues = value;
 | 
						|
}
 | 
						|
 | 
						|
static void
 | 
						|
cryptodev_backend_complete(UserCreatable *uc, Error **errp)
 | 
						|
{
 | 
						|
    CryptoDevBackend *backend = CRYPTODEV_BACKEND(uc);
 | 
						|
    CryptoDevBackendClass *bc = CRYPTODEV_BACKEND_GET_CLASS(uc);
 | 
						|
 | 
						|
    if (bc->init) {
 | 
						|
        bc->init(backend, errp);
 | 
						|
    }
 | 
						|
}
 | 
						|
 | 
						|
void cryptodev_backend_set_used(CryptoDevBackend *backend, bool used)
 | 
						|
{
 | 
						|
    backend->is_used = used;
 | 
						|
}
 | 
						|
 | 
						|
bool cryptodev_backend_is_used(CryptoDevBackend *backend)
 | 
						|
{
 | 
						|
    return backend->is_used;
 | 
						|
}
 | 
						|
 | 
						|
void cryptodev_backend_set_ready(CryptoDevBackend *backend, bool ready)
 | 
						|
{
 | 
						|
    backend->ready = ready;
 | 
						|
}
 | 
						|
 | 
						|
bool cryptodev_backend_is_ready(CryptoDevBackend *backend)
 | 
						|
{
 | 
						|
    return backend->ready;
 | 
						|
}
 | 
						|
 | 
						|
static bool
 | 
						|
cryptodev_backend_can_be_deleted(UserCreatable *uc)
 | 
						|
{
 | 
						|
    return !cryptodev_backend_is_used(CRYPTODEV_BACKEND(uc));
 | 
						|
}
 | 
						|
 | 
						|
static void cryptodev_backend_instance_init(Object *obj)
 | 
						|
{
 | 
						|
    /* Initialize devices' queues property to 1 */
 | 
						|
    object_property_set_int(obj, "queues", 1, NULL);
 | 
						|
}
 | 
						|
 | 
						|
static void cryptodev_backend_finalize(Object *obj)
 | 
						|
{
 | 
						|
    CryptoDevBackend *backend = CRYPTODEV_BACKEND(obj);
 | 
						|
 | 
						|
    cryptodev_backend_cleanup(backend, NULL);
 | 
						|
}
 | 
						|
 | 
						|
static void
 | 
						|
cryptodev_backend_class_init(ObjectClass *oc, void *data)
 | 
						|
{
 | 
						|
    UserCreatableClass *ucc = USER_CREATABLE_CLASS(oc);
 | 
						|
 | 
						|
    ucc->complete = cryptodev_backend_complete;
 | 
						|
    ucc->can_be_deleted = cryptodev_backend_can_be_deleted;
 | 
						|
 | 
						|
    QTAILQ_INIT(&crypto_clients);
 | 
						|
    object_class_property_add(oc, "queues", "uint32",
 | 
						|
                              cryptodev_backend_get_queues,
 | 
						|
                              cryptodev_backend_set_queues,
 | 
						|
                              NULL, NULL);
 | 
						|
}
 | 
						|
 | 
						|
static const TypeInfo cryptodev_backend_info = {
 | 
						|
    .name = TYPE_CRYPTODEV_BACKEND,
 | 
						|
    .parent = TYPE_OBJECT,
 | 
						|
    .instance_size = sizeof(CryptoDevBackend),
 | 
						|
    .instance_init = cryptodev_backend_instance_init,
 | 
						|
    .instance_finalize = cryptodev_backend_finalize,
 | 
						|
    .class_size = sizeof(CryptoDevBackendClass),
 | 
						|
    .class_init = cryptodev_backend_class_init,
 | 
						|
    .interfaces = (InterfaceInfo[]) {
 | 
						|
        { TYPE_USER_CREATABLE },
 | 
						|
        { }
 | 
						|
    }
 | 
						|
};
 | 
						|
 | 
						|
static void
 | 
						|
cryptodev_backend_register_types(void)
 | 
						|
{
 | 
						|
    type_register_static(&cryptodev_backend_info);
 | 
						|
}
 | 
						|
 | 
						|
type_init(cryptodev_backend_register_types);
 |