Exemple de code Source à partir de “noyau Linux programmation”

Il est très discutable. En fait, c'est carrément dangereux. Je vais donner à l'auteur le bénéfice du doute ici depuis qu'ils sont juste en essayant de montrer comment copy_from_user et copy_to_user travail, mais ils devraient vraiment ont fourni un exemple qui n'était pas si dangereux.

Surtout depuis le livre cires lyrique sur la façon dont vous doit être très prudent:

Appels système doit vérifier avec soin tous ses paramètres afin de s'assurer qu'ils sont valides et
juridique.L'appel système fonctionne dans le noyau de l'espace, et si l'utilisateur peut passer d'une entrée non valide dans le
noyau sans retenue, le système de sécurité et de stabilité peuvent en souffrir.

et fournit un moyen pour l'utilisateur d'annihiler complètement le noyau 🙂

---

Le texte de l'exemplaire que j'ai etats:

Prenons un exemple de l'appel système qui utilise à la fois copy_from_user() et copy_to_user().Cette syscall, silly_copy(), est tout à fait inutile; il copie des données à partir de son premier paramètre dans sa seconde.C'est sous-optimal en ce qu'il comporte un intermédiaire et étrangers copier dans le noyau de l'espace pour aucun gain. Mais il permet d'illustrer ce point.

/*
* silly_copy - pointless syscall that copies the len bytes from
* ‘src’ to ‘dst’ using the kernel as an intermediary in the copy.
* Intended as an example of copying to and from the kernel.
*/
SYSCALL_DEFINE3(silly_copy,
                unsigned long *, src,
                unsigned long *, dst,
                unsigned long len)
{
    unsigned long buf;

    /* copy src, which is in the user’s address space, into buf */
    if (copy_from_user(&buf, src, len))
        return -EFAULT;

    /* copy buf into dst, which is in the user’s address space */
    if (copy_to_user(dst, &buf, len))
        return -EFAULT;

    /* return amount of data copied */
    return len;
}

---

Autres que la défaillance catastrophique de ne pas avoir vérifié les paramètres, je suis assez certain que le dernier paramètre de la SYSCALL_DEFINE3 manque une virgule (même si c'était juste une faute de frappe).

Un bien meilleur exemple, sans avoir à allouer de la mémoire arbitraire, serait le long des lignes de:

SYSCALL_DEFINE3(silly_copy,
                unsigned long *, src,
                unsigned long *, dst,
                unsigned long,   len)
{
    unsigned long buf[64];                 /* Buffer for chunks */
    unsigned long lenleft = len;           /* Remaining size */
    unsigned long chunklen = sizeof(buf);  /* Initial chunk length */

    /* Loop handling chunk sizes */

    while (lenleft > 0) {
        /* Change chunk length on last chunk */

        if (lenleft < chunklen) chunklen = lenleft;

        /* copy src(user) to buf(kernel) then dst(user) */

        if (copy_from_user(buf, src, chunklen)) return -EFAULT;
        if (copy_to_user(dst, buf, chunklen)) return -EFAULT;

        /* Adjust pointers and remaining size */

        src += chunklen; dst += chunklen; lenleft -= chunklen;
    }

    /* return amount of data copied */
    return len;
}

---

Quelqu'un essaie de mettre en place ce système d'appel serait bien avisé de se tenir à l'écart à partir de cet exemple particulier, dans le livre, bien que je suppose que, au strict minimum, il vous donnera quelques bonnes débogage du noyau de l'expérience 🙂

OriginalL'auteur paxdiablo