search

n76 / DejaVu

Yet another network location backend for the UnifiedNLP/microG project
GNU General Public License v3.0
100 stars 18 forks source link

Blacklist compares mixed case with lower case #44

Open Helium314 opened 2 years ago

Helium314 commented 2 years ago

https://github.com/n76/DejaVu/blob/e54aae2d9f3d4619065a258b431ace1911d1e931/app/src/main/java/org/fitchfamily/android/dejavu/RfEmitter.java#L611 and https://github.com/n76/DejaVu/blob/e54aae2d9f3d4619065a258b431ace1911d1e931/app/src/main/java/org/fitchfamily/android/dejavu/RfEmitter.java#L686 results in "CDWiFi" being compared to "cdwifi" when the entry should match, so this entry is useless (applies to others as well).

Proposed solution: only use lowercase when comparing with lc.