Re: A possible bug and a fix (patch attached)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Oct 19, 2019 at 06:52:26PM -0400, M. Ranganathan wrote:
> I ran into an issue in config_file.c
> 
> The function is
> 
> static void wpa_config_write_network(FILE *f, struct wpa_ssid *ssid)
> 
> When writing out the ssid, the quote is omitted. I had to make the
> following changes:
> 
> #define QUOTED_STR(t) write_quoted_str(f, #t, ssid)
> 
>     //STR(ssid);
>     QUOTED_STR(ssid);
> 
> And I had to add the function :
> 
> static void write_quoted_str(FILE *f, const char *field, struct wpa_ssid *ssid)
> {
>     char *value = wpa_config_get(ssid, field);
>     if (value == NULL)
>         return;
>     fprintf(f, "\t%s=\"%s\"\n", field, value);
>     str_clear_free(value);
> }

This would break wpa_supplicant configuration writing for me..
wpa_config_get() returns a string with the needed quotation marks. Could
you please provide the exact parameter value in the ssid field that does
not work for you?

> Also I've run into a bug with dpp which can't handle SSID's larger
> than 4 characters it appears.

I have not seen this. Could you please provide more details and a debug
log showing this?
 
-- 
Jouni Malinen                                            PGP id EFC895FA

_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux