Copiar una string en otra, C (gcc)

Language: Layout:

//gcc 4.9.3

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define    STR_LEN    4    // El tamaño del string, más el caracter nulo '\0'

void str2p (char[], char*, int );    // Asigna los char de char[] en la variable apuntada por char*
//char* duplicar ( char* );
char* duplicar ( char* );      // Retorna una copia de los char del argumento char*

int main()
{
    char *p,*v;
  
  
    // Reserva memoria para el string.
    v = (char*)malloc(STR_LEN * sizeof(char));
    
    // Asigna la constante string "sip" a la variable v.
    str2p("sip123", v, STR_LEN);
    // Nótese que si la constante fuese más extensa,
    // por ejemplo "sip123", sólo las primeras
    // STR_LEN-1 letras podrán asignarse.
    
    
    
//    v="sip";
    
    printf("Variable v antes de free(v) = '%s'\n", v);
    printf("Variable p antes de free(v) = '%s'\n", p);
    
//p=duplicar(v);
    
    p = duplicar (v);
    
    
    free(v);                             //con el free no funciona ?poqueÂ¿
    
    printf("Variable v después de free(v) = '%s'\n", v);
    printf("Variable p después de free(v) = '%s'\n", p);
    
  
    return 0;
}

void str2p (char c[], char* v, int len)
{   
    int t = strlen(c) + 1;    // +1 incluye el caracter nulo '\0'
    
    // Para evitar desbordes se elige la longitud menor.
    int menor = ((t < len) ? t : len) - 1;
    
    // Comienza la asignación, letra por letra.
    for (int i=0; i < menor; i++)    v[i] = c[i];            
    
    // En C es obligatorio que todas las strings terminen en un caracter nulo '\0'
    v[menor] = 0;
}

char* duplicar (char* origen)
{
    char* k;
    int l = strlen(origen) + 1;    // Recordar siempre agregar el +1 para el caracter nulo '\0'
    
    if ( l == 0 )    k = NULL;
    else             k = (char*)malloc(l*sizeof(char)); //si malloc falla retorna NULL
    
    if (k == NULL) return NULL;
    
    for ( int i = 0; i < l-1; i++ )    k[i] = origen[i];

return k;
}

/*
char* duplicar( char* v )
{
    int i,     l = strlen(v);
    char *k;    
    
    if ( l == 0 )    k = NULL;
    else             k = (char*)malloc(l*sizeof(char)); //si malloc falla retorna NULL
    
    if (k == NULL) return NULL;
    
    for ( i = 0; i < l; i++ )
    {
        *( k + i ) = *( v + i );
    }
    return k;
}
*/

Show compiler warnings [ + ] Compiler args [ + ] Show input
Compilation time: 0.12 sec, absolute running time: 0.14 sec, cpu time: 0 sec, memory peak: 3 Mb, absolute service time: 0,34 sec