Print

Print


~~~
.../src/XrdCms/XrdCmsUtils.cc: In function 'XrdCmsUtils::ParseMan(XrdSysError*, XrdOucTList**, char*, char*, int*, bool)':
.../src/XrdCms/XrdCmsUtils.cc:226:43: error: pointer 'newMans_175' may be used after 'operator delete(void*, unsigned long)' [-Werror=use-after-free]
  226 |          if (!plus || strcmp(hSpec, newP->text)) isBad = false;
      |                                     ~~~~~~^~~~
.../src/XrdCms/XrdCmsUtils.cc:221:27: note: call to 'operator delete(void*, unsigned long)' here
  221 |                    delete newP;
      |                           ^~~~

.../src/XrdCl/XrdClCopy.cc: In function 'IndexRemote(XrdCl::FileSystem*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, unsigned short)':
.../src/XrdCl/XrdClCopy.cc:442:17: error: pointer used after 'operator delete(void*, unsigned long)' [-Werror=use-after-free]
  442 |       log->Error( AppMsg, "Bad URL: %s", current->Path );
      |       ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
.../src/XrdCl/XrdClCopy.cc:441:14: note: call to 'operator delete(void*, unsigned long)' here
  441 |       delete current;
      |              ^~~~~~~
~~~

You can view, comment on, or merge this pull request online at:

  https://github.com/xrootd/xrootd/pull/1598

-- Commit Summary --

  * Fix compiler warnings about pointers used after free:

-- File Changes --

    M src/XrdCl/XrdClCopy.cc (2)
    M src/XrdCms/XrdCmsUtils.cc (18)

-- Patch Links --

https://github.com/xrootd/xrootd/pull/1598.patch
https://github.com/xrootd/xrootd/pull/1598.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/pull/1598
You are receiving this because you are subscribed to this thread.

Message ID: <[log in to unmask]>

########################################################################
Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1