Fix reading from file cache using address instead of resolved address for getting a section

[augusto2112]

Summary

This fiex a small problem with https://reviews.llvm.org/D100338 where we'd grab the section from the address instead of resolved address, which could incur a memory read instead of a file cache read if the two addressess were different

[jasonmolenda]

Ach, I should have noticed that during review.

lldb/source/Target/Target.cpp

 //===-- Target.cpp --------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
 #include "lldb/Target/Target.h"
 #include "Plugins/ExpressionParser/Clang/ClangModulesDeclVendor.h"
 #include "lldb/Breakpoint/BreakpointIDList.h"
 #include "lldb/Breakpoint/BreakpointPrecondition.h"
 #include "lldb/Breakpoint/BreakpointResolver.h"
 #include "lldb/Breakpoint/BreakpointResolverAddress.h"
 #include "lldb/Breakpoint/BreakpointResolverFileLine.h"
 #include "lldb/Breakpoint/BreakpointResolverFileRegex.h"
 #include "lldb/Breakpoint/BreakpointResolverName.h"
if (!addr.IsSectionOffset()) {
     }
   }
   if (!resolved_addr.IsValid())
     resolved_addr = addr;
 
   bool is_readonly = false;
   // Read from file cache if read-only section.
   if (!force_live_memory && resolved_addr.IsSectionOffset()) {
-    SectionSP section_sp(addr.GetSection());
+    SectionSP section_sp(resolved_addr.GetSection());
     if (section_sp) {
       auto permissions = Flags(section_sp->GetPermissions());
       is_readonly = !permissions.Test(ePermissionsWritable) &&
                     permissions.Test(ePermissionsReadable);
     }
     if (is_readonly) {
       bytes_read = ReadMemoryFromFileCache(resolved_addr, dst, dst_len, error);
       if (bytes_read > 0)