14-隐私泄露缺陷

隐私泄露缺陷 - 腾讯安全代码审计实战系列14

隐私泄露发生在用户的敏感信息，如私人信息进入程序后，被错误地记录或传输到不安全的外部位置，例如文件系统、日志或网络。由于程序员对程序运行环境的盲目信任，可能导致这些信息被非授权人员获取。

修复建议

1. 避免将私人信息以明文格式记录在日志或外部文件中。
2. 在必要情况下，使用加密技术存储和传输敏感信息，以保护其隐私和完整性。
3. 实现数据脱敏处理，确保在日志或其他输出中不直接显示敏感信息。
4. 除非出于明确的业务需求，并且采取了适当的保护措施，否则不允许应用程序记录或传输私人信息。
5. 遵循最小授权原则，限制对敏感数据的访问权限，确保只有在职责范围内的人员或进程可以访问。

示例代码

//bad：直接将用户信息记录到日志，可能导致隐私泄露
public class LoggerBad {
    public void LogSensitiveData(string username, string password) {
        string logMessage = $"User Access - Username: {username}, Password: {password}";
        System.IO.File.AppendAllText("logs.txt", logMessage);  // 明文记录敏感信息
    }
}

//good：在记录日志前对敏感信息进行脱敏处理
public class LoggerGood {
    public void LogSensitiveDataWithMasking(string username, string password) {
        string maskedPassword = new String('*', password.Length);
        string logMessage = $"User Access - Username: {username}, Password: {maskedPassword}";
        System.IO.File.AppendAllText("secure_logs.txt", logMessage);  // 使用掩码替代真实密码
    }
}

//good：使用加密技术保护敏感数据后记录
public class LoggerWithEncryption {
    public void LogSensitiveDataEncrypted(string username, string password) {
        string encryptedPassword = Encrypt(password);  // 加密密码
        string logMessage = $"User Access - Username: {username}, Password: {encryptedPassword}";
        System.IO.File.AppendAllText("encrypted_logs.txt", logMessage);  // 加密后记录
    }

    private string Encrypt(string data) {
        // 示例加密方法，实际应用中使用强而有效的加密算法
        byte[] dataBytes = System.Text.Encoding.UTF8.GetBytes(data);
        return Convert.ToBase64String(dataBytes);
    }
}