>>      /*
>>       * Allow INSTALL_BIN_PATH to be used as a target path for the
>> @@ -213,20 +214,39 @@ void setup_consumerd_path(void)
>>       */
>>      path = getenv("LTTNG_CONSUMERD32_PATH");
>>      if (path) {
>> -            consumerd32_path = path;
>> +            ret = asprintf((char**)&consumerd32_path, "%s/" CONSUMERD_FILE, 
>> path);
> This cast tells me something is wrong. I'll let David reply.
>
> Thanks,
>
> Mathieu
>

In fact David suggested this ;)
Perhaps a "cleaner" option would be to use

char *tmp_path;
asprintf(&tmp_path,...);
consumerd32_path = tmp_path;

but using the cast avoids the need for a temporary variable.

_______________________________________________
lttng-dev mailing list
[email protected]
http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Reply via email to