Una posible solución.
Lo primero un escenario:
Create table Ventas (FechaVentas date, otraColumna int)
Go
Insert into Ventas (FechaVentas, otraColumna)
values
(GETDATE(), 1),
(GETDATE(), 2);
go
Con una tabla creada, modificamos el procedimiento, en varios pequeños detalles:
CREATE PROC variable3
@cn nvarchar(30),
@tb nvarchar(30)
As
BEGIN
DECLARE @sql nvarchar(200)
SET @sql = N'SELECT ' + QUOTENAME(@cn) + N' FROM ' + QUOTENAME(@tb) + N';';
END;
Exec sp_executeSql @sql;
GO
La variable @sql es de tipo nvarchar, no porque sea obligatorio, sino porque el procedimiento almacenado sp_executeSQL, recibe una variable de este tipo, y por tanto todos los literales, que se concatenan a la variable son N
.
Además se incluye la ejecución dinámica de la sentencia.
EXECUTE variable3 @cn = 'FECHAVENTAS',@tb = 'VENTAS'
La ejecución del mismo produce.

Normalmente cuando trabajes con código SQL Dínamico, prueba primero a incluir en vez de Exec sp....
, un print @sql
. Luego mandas ejecutar el procedure y te devolverá la sentencia que intentará ejecutar. Esto te provee de más información.
Si observas realmente la sentencia, he incluido espacios entre la columna y From y la tabla, y he terminado la sentencia con un punto y coma.
Además te recomiendo que intentes parametrizar la ejecución del procedimiento sp_executeSQL
Parametrizar exec sp_ExecuteSql
EXEC(@sql)
osp_executesql @sql
después delSET
. Ahora, a menos que tengas control absoluto de cómo se le pasan variables a sp o sea sólo para uso interno, esto no es recomendable, te deja abierto a sql injection