Advertencia: "el formato no es una cadena literal ni argumentos de formato"

Desde que actualicé a la última versión de Xcode 3.2.1 y Snow Leopard, he recibido la advertencia

"formato no es una cadena literal ni argumentos de formato"

del siguiente código:

NSError *error = nil;

if (![self.managedObjectContext save:&error]) 
{
    NSLog([NSString stringWithFormat:@"%@ %@, %@", 
       errorMsgFormat, 
       error, 
       [error userInfo]]);      

}

Si errorMsgFormates un NSStringformato con especificadores (por ejemplo:) "print me like this: %@", ¿qué hay de malo en la NSLogllamada anterior ? ¿Y cuál es la forma recomendada de solucionarlo para que no se genere la advertencia?

¿Está anidando sus soportes correctamente? No creo que NSLog()le guste tomar solo un argumento, que es lo que le está pasando. Además, ya realiza el formateo por ti. ¿Por qué no hacer esto?

NSLog(@"%@ %@, %@", 
   errorMsgFormat, 
   error, 
   [error userInfo]);              

O, como dice que errorMsgFormates una cadena de formato con un único marcador de posición, ¿está intentando hacer esto?

NSLog(@"%@, %@", [NSString stringWithFormat:errorMsgFormat, error], 
   [error userInfo]);              

Xcode se queja porque se trata de un problema de seguridad.

Aquí hay un código similar al tuyo:

NSString *nameFormat = @"%@ %@";
NSString *firstName = @"Jon";
NSString *lastName = @"Hess %@";
NSString *name = [NSString stringWithFormat:nameFormat, firstName, lastName];
NSLog(name);

Esa última declaración NSLog ejecutará el equivalente a esto:

NSLog(@"Jon Hess %@");

Eso hará que NSLog busque un argumento de cadena más, pero no hay uno. Debido a la forma en que funciona el lenguaje C, tomará un puntero de basura aleatorio de la pila e intentará tratarlo como un NSString. Lo más probable es que esto bloquee su programa. Ahora tus cadenas probablemente no tengan% @ en ellas, pero algún día podrían tenerlas. Siempre debe usar una cadena de formato con datos que controle explícitamente como primer argumento para las funciones que toman cadenas de formato (printf, scanf, NSLog, - [NSString stringWithFormat:], ...).

Como señala Otto, probablemente deberías hacer algo como:

NSLog(errorMsgFormat, error, [error userInfo]);

Respuesta final: como dijo Jon Hess, es un problema de seguridad porque está pasando una cadena CUALQUIER a una función que espera una cadena de formato. Es decir, evaluará todos los especificadores de formato DENTRO de la cadena. Si no los hay, genial, pero si los hay, pueden pasar cosas malas.

Entonces, lo correcto es USAR una cadena de formato directamente, por ejemplo

NSLog(@"%@", myNSString);

De esa manera, incluso si hay especificadores de formato en myNSString, NSLog no los evalúa.

No recomiendo especialmente usar esto, ya que la advertencia ES una advertencia real ... en un uso dinámico del lenguaje es posible hacer cosas en tiempo de ejecución de la cadena (es decir, insertar nueva información o incluso bloquear el programa). Sin embargo, es posible forzar la supresión si SABES que debería ser así y realmente no quieres que te adviertan sobre ello.

#pragma GCC diagnostic ignored "-Wformat-security"

Le diría a GCC que ignore temporalmente la advertencia de compilación. Una vez más, no resuelve nada, pero puede haber ocasiones en las que no pueda encontrar una buena manera de solucionar el problema.

EDITAR: A partir de clang, el pragma ha cambiado. Vea esto: https://stackoverflow.com/a/17322337/3937

La forma más rápida de solucionarlo sería agregar @"%@",como primer argumento a su NSLogllamada, es decir,

NSLog(@"%@", [NSString stringWithFormat: ....]);

Sin embargo, probablemente deberías considerar la respuesta de Sixteen Otto.

Acabo de pasar un cero para negar las advertencias, ¿tal vez eso funcione para usted?

NSLog (myString, nulo);

Si desea deshacerse de la advertencia "formato, no una cadena literal y sin argumentos de formato" de una vez por todas, puede desactivar la configuración de advertencia de GCC "Typecheck Calls to printf / scanf" (GCC_WARN_TYPECHECK_CALLS_TO_PRINTF = NO) en la configuración de compilación de su objetivo.

NSLog () espera una cadena de formato, lo que se pasa es solo una cadena. No es necesario utilizar stringWithFormat :, solo puede hacer:

NSLog(@"%@ %@, %@", errorMsgFormat, error, [error userInfo])

Y eso haría que la advertencia desapareciera.

FWIW, esto también se aplica a los desarrolladores de iPhone. Estoy codificando contra el SDK 3.1.3 y obtuve el mismo error con el mismo problema (anidando stringWithFormat dentro de NSLog ()). Sixten y Jon están en el dinero.

El solo hecho appendFormatde informar a cualquiera que use on NSMutableString también puede hacer que aparezca esta advertencia si intenta pasar una cadena formateada como esta:

NSMutableString *csv = [NSMutableString stringWithString:@""];
NSString *csvAddition = [NSString stringWithFormat:@"%@",WHATEVERYOUAREPUTTINGINYOURSTRING];
[csv appendFormat:csvAddition];

Entonces, para evitar esta advertencia, convierta lo anterior en esto:

NSMutableString *csv = [NSMutableString stringWithString:@""];
[csv appendFormat:@"%@",WHATEVERYOUAREPUTTINGINYOURSTRING];

Más conciso y seguro. ¡Disfrutar!

NSLog(@"%@ %@, %@", 
       errorMsgFormat, 
       error, 
       [error userInfo]);