c++ - Why is this COM code leaking? -


i'm maintaining application uses windows explorer overlay icons. operations require me forcibly refresh explorers view particular folder. using following function uses com:

void refreshexplorerview(cstring strpath) {     ccomptr<ishellwindows> pshellwindows;      coinitialize(null);      if(succeeded(pshellwindows.cocreateinstance(clsid_shellwindows)))     {         idispatch* pfolder=null;         variant variant;         v_vt(&variant) = vt_i4;          for(v_i4(&variant) = 0; pshellwindows->item(variant, &pfolder) == s_ok; v_i4(&variant)++)         {             ccomptr<iwebbrowserapp> pwebbrowserapp;             if(succeeded(pfolder->queryinterface(iid_ppv_args(&pwebbrowserapp))))             {                 bstr locationurl = null;                 pwebbrowserapp->get_locationurl(&locationurl);                  if(locationurl != null && strpath.comparenocase(locationurl) == 0)                 {                     ccomptr<iserviceprovider> pserviceprovider;                     if(succeeded(pwebbrowserapp->queryinterface(iid_ppv_args(&pserviceprovider))))                     {                         ccomptr<ishellbrowser> pshellbrowser;                         if(succeeded(pserviceprovider->queryinterface(iid_ppv_args(&pshellbrowser))))                         {                             ishellview* pshellview;                             if(succeeded(pshellbrowser->queryactiveshellview(&pshellview)))                             {                                 pshellview->refresh();                                 pshellview->release();                             }                         }                     }                 }                  sysfreestring(locationurl);             }             pfolder->release();             pfolder = null;         }     }      couninitialize(); }

i've noticed when program refresh regularly grows in size , umdh has shown me appear leaking pfolder , pshellwindow instances every time runs. can't work out why on earth happens since far can tell these released properly. can see i'm missing?

you release pshellwindows after couninitialize, incorrect.

the rest of interfaces seem released fine. note improve cleanless , readability greatly using ccomqiptr instead of queryinterface, , not using raw pointers @ (bstr, ifoo*) , replace them smart auto-releasing wrappers.

pfolder might leaking too, if item call successful returns code other s_ok. again, use of ccomptr<ifolder> instead of ifolder* resolve problem without drawing attention it.


Comments

Post a Comment

Popular posts from this blog

php - Why I am getting the Error "Commands out of sync; you can't run this command now" -

the documentation of error mentioned in title says if commands out of sync; can't run command now in client code, calling client functions in wrong order. this can happen, example, if using mysql_use_result() , try execute new query before have called mysql_free_result(). can happen if try execute 2 queries return data without calling mysql_use_result() or mysql_store_result() in between. from here: http://dev.mysql.com/doc/refman/5.0/en/commands-out-of-sync.html but in first query not fetching data mysql database, inserting. , in second query getting data database. here code $connection = mysqli_connect("localhost","username","password","tbl_msgs"); if(mysqli_connect_errno($connection)) { die("failed connect mysql: " . mysqli_connect_error()); } $query = "insert users (total_comments, total_views) values ({$total_comments}, {$total_views});"; $query .= "insert msgs (no...
Read more

linux - Does gcc have any options to add version info in ELF binary file? -

i mean whether gcc can insert source code version infor elf binary section or similar. not want change source file, add info gcc option in makefile. you can emit version info text file, turn text file object file statically link executable. the first step simple have write code: script or write version info in format plain text file. write makefile rule produce version.o version.txt, using objcopy . you'll have object file 2 useful symbols defined in it: beginning , end of textual version info. add generated object executable, , you'll able access version 2 ways: running strings on binary, or writing code in application print version string (you'll need declare start , end symbols variables in header file).
Read more

java - Are there any classes that implement javax.persistence.Parameter<T>? -

i'm using named queries fetch entities database using jpa , hibernate , came across puzzling me: how come there no default implementations of javax.persistence.parameter interface? i rather impressed how @staticmetamodel used criteriabuilder in jpa2 meaning type safety , don't have mess around setting string names. wanted apply similar of jpa1 ejb's i'm working on: dealergroup_.class: public dealergroup_ { public parameter<integer> id = new parameterimpl<integer>("id"); /// etc... } dealergroupfacade.class: // ... public dealergroup fetch(integer id) { typedquery<dealergroup> query = em.createnamedquery("dealergroup.fetchbyid", dealergroup.class); query.setparameter(dealergroup_.id, query); return query.getsingleresult(); } // ... i surprised find there no implementations of parameter interface - of course, implement myself absence makes me suspicious may mistake using in way? should implementing parameter...
Read more